casra-developers commented on pull request #16110:
URL: https://github.com/apache/airflow/pull/16110#issuecomment-853612966
I try to narrow it down. Luckily, it seems to be mostly two files that
require those dependencies in a limited manner:
- airflow/utils/process_utils.py
- **termios**: On line 159 in execute_interactive
- **termios**: On line 180 in execute_interactive
- **tty**: On line 160 in execute_interactive
- **pty**: On line 163 in execute_interactive
- airflow/utils/timeout.py
- **signal**: On line 40 in __enter__
- **signal**: On line 41in __enter__
- **signal**: On line 47 in __exit__
In "process_utils.py" we probably can just use IS_WINDOWS to not use the
missing features. The timeout one is probably a bit more tricky although we
could still use the thread-based Windows implementation without hiding it
behind the modified signal module.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
