Move manager initialization outside of top level scope. Fixes #233 (#236)

.gitignore

@@ -18,7 +18,7 @@ proxy.py.iml
 *.crt
 *.key
 
-venv
+venv*
 cover
 htmlcov
 dist

proxy/core/event.py

@@ -37,28 +37,16 @@ eventNames = EventNames(1, 2, 3, 4, 5, 6, 7, 8)
 
 
 class EventQueue:
-    """Global event queue."""
+    """Global event queue.
 
-    MANAGER: multiprocessing.managers.SyncManager = multiprocessing.Manager()
-
-    def __init__(self) -> None:
-        super().__init__()
-        self.queue = EventQueue.MANAGER.Queue()
-
-    def publish(
-        self,
-        request_id: str,
-        event_name: int,
-        event_payload: Dict[str, Any],
-        publisher_id: Optional[str] = None
-    ) -> None:
-        """Publish an event into the queue.
+    Each event contains:
 
     1. Request ID               - Globally unique
     2. Process ID               - Process ID of event publisher.
                                   This will be process id of acceptor workers.
     3. Thread ID                - Thread ID of event publisher.
-                                      When --threadless is enabled, this value will be same for all the requests
+                                  When --threadless is enabled, this value will
+                                  be same for all the requests
                                   received by a single acceptor worker.
                                   When --threadless is disabled, this value will be
                                   Thread ID of the thread handling the client request.
@@ -67,6 +55,19 @@ class EventQueue:
     6. Event Payload            - Optional data associated with the event
     7. Publisher ID (optional)  - Optionally, publishing entity unique name / ID
     """
+
+    def __init__(self) -> None:
+        super().__init__()
+        self.manager = multiprocessing.Manager()
+        self.queue = self.manager.Queue()
+
+    def publish(
+        self,
+        request_id: str,
+        event_name: int,
+        event_payload: Dict[str, Any],
+        publisher_id: Optional[str] = None
+    ) -> None:
         self.queue.put({
             'request_id': request_id,
             'process_id': os.getpid(),
@@ -169,9 +170,8 @@ class EventDispatcher:
 class EventSubscriber:
     """Core event subscriber."""
 
-    MANAGER: multiprocessing.managers.SyncManager = multiprocessing.Manager()
-
     def __init__(self, event_queue: EventQueue) -> None:
+        self.manager = multiprocessing.Manager()
         self.event_queue = event_queue
         self.relay_thread: Optional[threading.Thread] = None
         self.relay_shutdown: Optional[threading.Event] = None
@@ -180,7 +180,7 @@ class EventSubscriber:
 
     def subscribe(self, callback: Callable[[Dict[str, Any]], None]) -> None:
         self.relay_shutdown = threading.Event()
-        self.relay_channel = EventSubscriber.MANAGER.Queue()
+        self.relay_channel = self.manager.Queue()
         self.relay_thread = threading.Thread(
             target=self.relay,
             args=(self.relay_shutdown, self.relay_channel, callback))