RNHTTR commented on PR #35097:
URL: https://github.com/apache/airflow/pull/35097#issuecomment-1773804072

   I did add this blurb about imports:
   
   > Note that import statements also count as top-level code. So, if you have 
an import statement that takes a long time or the imported module itself 
executes code at the top-level, that too can impact the performance of the 
scheduler.
   
   Even the pandas example feels like it'd be a relatively small optimization 
compared to the kind of problems that more significant top level code can 
cause, and IMO including that example could cause more confusion than good for 
folks who are newer to Airflow. For example, this PR was inspired by [this 
stackoverflow question](https://stackoverflow.com/q/77332542/7004653).


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

Reply via email to