o-nikolas commented on code in PR #33481:
URL: https://github.com/apache/airflow/pull/33481#discussion_r1303264195
##########
airflow/cli/cli_parser.py:
##########
@@ -62,6 +63,13 @@
# Do no re-raise the exception since we want the CLI to still function for
# other commands.
+try:
+ auth_mgr = get_auth_manager()
Review Comment:
@vandonr-amz Did you end up looking at the affect of the import time of the
fab auth manager (second paragraph in my first comment)?
I think it's important to do at least some quick benchmarking to see how
slow or fast it is. Before I optimized the executors they added seconds to the
CLI runtime just because loading the module was slow.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
