tommalt commented on PR #52091: URL: https://github.com/apache/airflow/pull/52091#issuecomment-3000755064
Yes, it is necessary. In the documentation you linked, it reads: > it [zipimport library] is automatically used by the built-in import mechanism for sys.path items *that are paths to ZIP archives.* What this means is that in order for `zipimport` to work, you need an entry in sys.path which points to the zip archive in question. This is [demonstrated in the example](https://docs.python.org/3/library/zipimport.html#examples), which is the same thing that I am doing here. It is also what the airflow scheduler does in order to handle zip archives: This is how the scheduler currently handles zip files, from which I derived my triggerer implementation. https://github.com/apache/airflow/blob/e1d9be482ac567d038c9f78d89f0a7c944139585/airflow-core/src/airflow/models/dagbag.py#L468-L516 -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
