[GitHub] [airflow] pcolladosoto commented on issue #22191: dag_processing code needs to handle OSError("handle is closed") in poll() and recv() calls

2022-04-01 Thread GitBox
pcolladosoto commented on issue #22191: URL: https://github.com/apache/airflow/issues/22191#issuecomment-1085746755 Hi! As seen above I've just opened a related PR: #22685 It would be great if we could continue the discussion over there and iterate over a few solutions until we

[GitHub] [airflow] pcolladosoto commented on issue #22191: dag_processing code needs to handle OSError("handle is closed") in poll() and recv() calls

2022-04-01 Thread GitBox
pcolladosoto commented on issue #22191: URL: https://github.com/apache/airflow/issues/22191#issuecomment-1085691641 That's one if the issues I wanted to bring up on the PR... As you say, `SIGKILL` *should* terminate the process in any circumstance. However, I do lean towards adding an

[GitHub] [airflow] pcolladosoto commented on issue #22191: dag_processing code needs to handle OSError("handle is closed") in poll() and recv() calls

2022-03-27 Thread GitBox
pcolladosoto commented on issue #22191: URL: https://github.com/apache/airflow/issues/22191#issuecomment-1080033434 Hi @potiuk! I'm glad you found our investigation useful and that you had fun reading through it. Reading so may Agatha Christie books has to pay off at some point  I

[GitHub] [airflow] pcolladosoto commented on issue #22191: dag_processing code needs to handle OSError("handle is closed") in poll() and recv() calls

2022-03-21 Thread GitBox
pcolladosoto commented on issue #22191: URL: https://github.com/apache/airflow/issues/22191#issuecomment-1074273533 # Trying to explain things... Our team has run into this issue time and time again. We have tried different combinations of both Airflow and Python versions to no avail.