From 3cf1d6437247faf2629330847c1da42f696b447b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicolas=20=C3=89vrard?= <nicoe@b2ck.com>
Date: Wed, 8 Jan 2025 23:20:32 +0100
Subject: [PATCH] Use process-local data structure to store process related
 data

---
 trytond/trytond/cache.py            | 34 ++++++++++++++++-------------
 trytond/trytond/tests/test_cache.py |  4 +---
 2 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/trytond/trytond/cache.py b/trytond/trytond/cache.py
index 4ba831ce43..6945411a9f 100644
--- a/trytond/trytond/cache.py
+++ b/trytond/trytond/cache.py
@@ -3,7 +3,6 @@
 import datetime as dt
 import json
 import logging
-import os
 import selectors
 import threading
 from collections import OrderedDict, defaultdict
@@ -18,6 +17,7 @@ from trytond import backend
 from trytond.config import config
 from trytond.pool import Pool
 from trytond.tools import grouped_slice, resolve
+from trytond.tools.multiprocessing import local
 from trytond.transaction import Transaction
 
 __all__ = ['BaseCache', 'Cache', 'LRUDict', 'LRUDictTransaction']
@@ -64,6 +64,13 @@ def _get_modules(cursor):
     return {m for m, in cursor}
 
 
+class _CacheLocal(local):
+
+    def __init__(self):
+        self.listeners = {}
+        self.listener_lock = threading.Lock()
+
+
 class BaseCache(object):
     _instances = {}
     context_ignored_keys = {
@@ -155,8 +162,7 @@ class MemoryCache(BaseCache):
     _reset = WeakKeyDictionary()
     _clean_last = None
     _default_lower = Transaction.monotonic_time()
-    _listener = {}
-    _listener_lock = defaultdict(threading.Lock)
+    _local = _CacheLocal()
     _table = 'ir_cache'
     _channel = _table
 
@@ -238,11 +244,11 @@ class MemoryCache(BaseCache):
         database = transaction.database
         dbname = database.name
         if not _clean_timeout and database.has_channel():
-            pid = os.getpid()
-            with cls._listener_lock[pid]:
-                if (pid, dbname) not in cls._listener:
-                    cls._listener[pid, dbname] = listener = threading.Thread(
+            with cls._local.listener_lock:
+                if dbname not in cls._local.listeners:
+                    listener = threading.Thread(
                         target=cls._listen, args=(dbname,), daemon=True)
+                    cls._local.listeners[dbname] = listener
                     listener.start()
             return
         last_clean = (dt.datetime.now() - cls._clean_last).total_seconds()
@@ -344,9 +350,8 @@ class MemoryCache(BaseCache):
 
     @classmethod
     def drop(cls, dbname):
-        pid = os.getpid()
-        with cls._listener_lock[pid]:
-            listener = cls._listener.pop((pid, dbname), None)
+        with cls._local.listener_lock:
+            listener = cls._local.listeners.pop(dbname, None)
         if listener:
             database = backend.Database(dbname)
             conn = database.get_connection()
@@ -380,7 +385,6 @@ class MemoryCache(BaseCache):
     @classmethod
     def _listen(cls, dbname):
         current_thread = threading.current_thread()
-        pid = os.getpid()
 
         conn, selector = None, None
         try:
@@ -401,7 +405,7 @@ class MemoryCache(BaseCache):
             current_thread.listening = True
 
             selector.register(conn, selectors.EVENT_READ)
-            while cls._listener.get((pid, dbname)) == current_thread:
+            while cls._local.listeners.get(dbname) == current_thread:
                 selector.select(timeout=_select_timeout)
                 conn.poll()
                 while conn.notifies:
@@ -425,9 +429,9 @@ class MemoryCache(BaseCache):
                 selector.close()
             if conn:
                 database.put_connection(conn)
-            with cls._listener_lock[pid]:
-                if cls._listener.get((pid, dbname)) == current_thread:
-                    del cls._listener[pid, dbname]
+            with cls._local.listener_lock:
+                if cls._local.listeners.get(dbname) == current_thread:
+                    del cls._local.listeners[dbname]
 
 
 if config.get('cache', 'class'):
diff --git a/trytond/trytond/tests/test_cache.py b/trytond/trytond/tests/test_cache.py
index 29e89ac387..552d694d22 100644
--- a/trytond/trytond/tests/test_cache.py
+++ b/trytond/trytond/tests/test_cache.py
@@ -2,7 +2,6 @@
 # this repository contains the full copyright notices and license terms.
 
 import datetime as dt
-import os
 import time
 import unittest
 from unittest.mock import patch
@@ -256,9 +255,8 @@ class MemoryCacheChannelTestCase(MemoryCacheTestCase):
             time.sleep(.01)
 
     def wait_cache_listening(self):
-        pid = os.getpid()
         dbname = Transaction().database.name
-        listener = MemoryCache._listener.get((pid, dbname))
+        listener = MemoryCache._local.listeners[dbname]
         while (not getattr(listener, 'listening', False)
                 and listener.is_alive()):
             time.sleep(.01)