You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Taragolis (via GitHub)" <gi...@apache.org> on 2023/02/14 00:28:37 UTC

[GitHub] [airflow] Taragolis commented on pull request #29494: Fix circular imports when airflow starts

Taragolis commented on PR #29494:
URL: https://github.com/apache/airflow/pull/29494#issuecomment-1428915301

   Some time ago I tried to write specific tests for known and repeatable problem with circular imports.
   
   There is couple issues (I guess all could be solved):
   1. Python 3.7 only show as regular ImportError without mentioning that is circular import
   2. Need to run all this kind of tests outside of regular `pytest` environment because everything already imported in the right way. I found a stupid solution - generate python file and execute it in subprocess
   3. If run in subprocess need to mock specific stuff very carefully.


-- 
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