Taragolis commented on issue #35644:
URL: https://github.com/apache/airflow/issues/35644#issuecomment-1812305539

   > Yeah. But it's not the same issue (or at least I believe it's not) - I 
believe the original issue was connected with SIGALRM not being properly 
handled by long-running low-level c-code. 
   
   To be honest we don't know what the reason in particular this cases. It 
could be both:
   1. Redefine SIGALRM handlers in upstream code
   2. To broad catch exceptions in libraries or user code
   
   In second case when Airflow task SIGALARM works, we do not control in which 
place in Main thread code returns
   
   In this case it will break Airflow timeout, and in addition it might breaks 
something else
   ```python
   try
       ...  # <- returns here
   except Exception:
       pass
   ```
    
   In case if handler return in this code block, then change inheritance 
wouldn't help at all
   
   ```python
   try
       ...  # <- returns here
   except:  # or except BaseException:
       pass
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to