You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "shashbha14 (via GitHub)" <gi...@apache.org> on 2026/03/15 12:39:47 UTC
[PR] docs: clarify plugins folder sys.modules registration behavior [airflow]
shashbha14 opened a new pull request, #63634:
URL: https://github.com/apache/airflow/pull/63634
The Modules Management documentation currently assumes that the dags, config, and plugins folders are equivalent and
all get added to the sys.path list. This is incorrect because the plugins folder also imports all .py files in this directory
and registers them in the sys.modules namespace under the filename without the directory structure. This means that if you
have a file named hdfs.py in the plugins/my_project directory, the file will end up as sys.modules["hdfs"], which can
cause name clashes with other packages, such as the PyPI hdfs package. For this-
Two warning sections have been added:
- In "Built-in PYTHONPATH entries" - describing the behavior of the sys.modules registration.
- In "Use unique top package name" - describing the depth at which this can occur and the importance of the top-level
package name being unique.
This Fixes #63548
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] docs: clarify plugins folder sys.modules registration behavior [airflow]
Posted by "shashbha14 (via GitHub)" <gi...@apache.org>.
shashbha14 commented on PR #63634:
URL: https://github.com/apache/airflow/pull/63634#issuecomment-4065085391
The Static checks failure and the modification to output_pr_auto-triage.txt are unrelated to this documentation-only PR. They appeared after the merge commit from main. Could a maintainer please advise or re-run?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] docs: clarify plugins folder sys.modules registration behavior [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #63634:
URL: https://github.com/apache/airflow/pull/63634
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org