Avoid logging in __del__ functions (#76)

bodo-ai · Dec 19, 2024 · 827e909 · 827e909
1 parent 9c34d32
commit 827e909
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 20 deletions.
diff --git a/bodo/pandas/array_manager.py b/bodo/pandas/array_manager.py
@@ -261,10 +261,6 @@ def __del__(self):
         otherwise we do nothing because the data is already collected/deleted.
         """
         if (r_id := self._md_result_id) is not None:
-            debug_msg(
-                self.logger,
-                f"[LazyArrayManager] Asking workers to delete result '{r_id}'",
-            )
             assert self._del_func is not None
             self._del_func(r_id)
             self._del_func = None
@@ -501,10 +497,6 @@ def __del__(self):
         otherwise we do nothing because the data is already collected/deleted.
         """
         if (r_id := self._md_result_id) is not None:
-            debug_msg(
-                self.logger,
-                f"[LazySingleArrayManager] Asking workers to delete result '{r_id}'",
-            )
             assert self._del_func is not None
             self._del_func(r_id)
             self._del_func = None

diff --git a/bodo/pandas/arrow/array.py b/bodo/pandas/arrow/array.py
@@ -128,10 +128,6 @@ def __del__(self):
         Delete the result from workers if it exists.
         """
         if (r_id := self._md_result_id) is not None:
-            debug_msg(
-                self.logger,
-                f"[LazyArrowExtensionArray] Asking workers to delete result '{r_id}'",
-            )
             assert self._del_func is not None
             self._del_func(r_id)
             self._del_func = None

diff --git a/bodo/pandas/managers.py b/bodo/pandas/managers.py
@@ -244,10 +244,6 @@ def __del__(self):
         Delete the result from the workers if it hasn't been collected yet.
         """
         if (r_id := self._md_result_id) is not None:
-            debug_msg(
-                self.logger,
-                f"[LazyBlockManager] Asking workers to delete result '{r_id}'",
-            )
             assert self._del_func is not None
             self._del_func(r_id)
             self._del_func = None
@@ -437,10 +433,6 @@ def __del__(self):
         Delete the result from the workers if it hasn't been collected yet.
         """
         if (r_id := self._md_result_id) is not None:
-            debug_msg(
-                self.logger,
-                f"[LazySingleBlockManager] Asking workers to delete result '{r_id}'",
-            )
             assert self._del_func is not None
             self._del_func(r_id)
             self._del_func = None