Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
liyanzhe.lyz committed Jan 26, 2025
1 parent 645333a commit 8ad816d
Showing 1 changed file with 4 additions and 12 deletions.
16 changes: 4 additions & 12 deletions dlrover/python/training_event/error_handler.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,16 +90,9 @@ def _handle_signal(self, signum, frame):
_LOGGER.error(f"process signal {signum} error: {str(e)}")

finally:
# call original handler
original_handler = self._original_handlers[signum]
if callable(original_handler):
original_handler(signum, frame)
else:
if original_handler == signal.SIG_DFL:
signal.signal(signum, signal.SIG_DFL)
os.kill(os.getpid(), signum)
elif original_handler == signal.SIG_IGN:
pass
self.unregister()
# call original handler with signal
os.kill(os.getpid(), signum)

def _register(self):
"""register exception handler"""
Expand Down Expand Up @@ -138,8 +131,7 @@ def _unregister(self):
self._original_excepthook = None

for sig, handler in self._original_handlers.items():
if handler:
signal.signal(sig, handler)
signal.signal(sig, handler)
self._original_handlers.clear()

self._registered = False
Expand Down

0 comments on commit 8ad816d

Please sign in to comment.