process/Broker: do not reset poller_class. Fixes mitogen-hq#967

philfry · Apr 2, 2024 · fc40ec9 · fc40ec9
1 parent 301ae9c
commit fc40ec9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/ansible_mitogen/process.py b/ansible_mitogen/process.py
@@ -280,12 +280,7 @@ def get_cpu_count(default=None):
 
 
 class Broker(mitogen.master.Broker):
-    """
-    WorkerProcess maintains at most 2 file descriptors, therefore does not need
-    the exuberant syscall expense of EpollPoller, so override it and restore
-    the poll() poller.
-    """
-    poller_class = mitogen.core.Poller
+    pass
 
 
 class Binding(object):

diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -21,6 +21,7 @@ v0.3.5 (2024-03-17)
 -------------------
 
 * :gh:issue:`987` Support Python 3.11
+* :gh:issue:`967` Broker: Do not reset poller_class to mitogen.core.Poller to avoid 'filedescriptor out of range in select()'
 * :gh:issue:`885` Fix :py:exc:`PermissionError` in :py:mod:`importlib` when
   becoming an unprivileged user with Python 3.x
 * :gh:issue:`1033` Support `PEP 451 <https://peps.python.org/pep-0451/>`_,