fix task/thread name and delete commented out code

Author: sleepyStick

pymongo/asynchronous/monitor.py

@@ -136,9 +136,13 @@ def __init__(
         The Topology is weakly referenced. The Pool must be exclusive to this
         Monitor.
         """
+        if _IS_SYNC:
+            name = "pymongo_server_monitor_thread"
+        else:
+            name = "pymongo_server_monitor_task"
         super().__init__(
             topology,
-            "pymongo_server_monitor_task",
+            name,
             topology_settings.heartbeat_frequency,
             common.MIN_HEARTBEAT_INTERVAL,
         )

pymongo/asynchronous/topology.py

@@ -653,15 +653,7 @@ async def request_check_all(self, wait_time: int = 5) -> None:
         """Wake all monitors, wait for at least one to check its server."""
         async with self._lock:
             self._request_check_all()
-            # if _IS_SYNC:
             await self._condition.wait(wait_time)
-            # else:
-            #     try:
-            #         await asyncio.wait_for(
-            #             self._condition.wait(), wait_time
-            #         )  # type-ignore: [arg-type]
-            #     except asyncio.TimeoutError:
-            #         pass
 
     def data_bearing_servers(self) -> list[ServerDescription]:
         """Return a list of all data-bearing servers.

pymongo/synchronous/monitor.py

@@ -136,9 +136,13 @@ def __init__(
         The Topology is weakly referenced. The Pool must be exclusive to this
         Monitor.
         """
+        if _IS_SYNC:
+            name = "pymongo_server_monitor_task"
+        else:
+            name = "pymongo_server_monitor_task"
         super().__init__(
             topology,
-            "pymongo_server_monitor_task",
+            name,
             topology_settings.heartbeat_frequency,
             common.MIN_HEARTBEAT_INTERVAL,
         )

pymongo/synchronous/topology.py

@@ -651,15 +651,7 @@ def request_check_all(self, wait_time: int = 5) -> None:
         """Wake all monitors, wait for at least one to check its server."""
         with self._lock:
             self._request_check_all()
-            # if _IS_SYNC:
             self._condition.wait(wait_time)
-            # else:
-            #     try:
-            #         asyncio.wait_for(
-            #             self._condition.wait(), wait_time
-            #         )  # type-ignore: [arg-type]
-            #     except asyncio.TimeoutError:
-            #         pass
 
     def data_bearing_servers(self) -> list[ServerDescription]:
         """Return a list of all data-bearing servers.