You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/04/03 06:48:12 UTC

[GitHub] [airflow] tirkarthi commented on pull request #22613: Convert CLI commands to use Click

tirkarthi commented on pull request #22613:
URL: https://github.com/apache/airflow/pull/22613#issuecomment-1086788871


   The imports might need an optimization as I see noticeable slowdown and click based cli having more imports. You can use `importtime` to see the import tree as it's large to paste : https://docs.python.org/3/using/cmdline.html#envvar-PYTHONPROFILEIMPORTTIME . The timings are on Python 3.10
   
   ```
   (.env) ➜  airflow git:(cli-roles) ✗ python -X importtime -m airflow.cli 2>&1 | grep 'import time' | wc
      2764   19350  230153
   (.env) ➜  airflow git:(cli-roles) ✗ python -X importtime -m airflow 2>&1 | grep 'import time' | wc   
       614    4300   40655
   ```


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