potiuk commented on PR #29055: URL: https://github.com/apache/airflow/pull/29055#issuecomment-1480004738
> Thanks @potiuk, I'll rebase now. > > I have also noticed that since we now load the executors to retrieve CLI commands which they want to vend, it does slow down the time it takes to run a CLI command (importing the executors is expensive, they have quite a large dependency trees). I'll try collect some data today and post more information with this freshly rebased code. Yeah. I think the error was really indication of this problem (so in a way it's good it failed). I think we do not really import all of that before (we are doing a lot of lazy loading to speed up CLI commands). Maybe we should actually add some tests for CLIs being **fast** (@ashb @uranusjr WDYT? - you've been doing a lot of lazy loading recently and in the past ) -- 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]
