jason810496 commented on issue #46978: URL: https://github.com/apache/airflow/issues/46978#issuecomment-2680295342
> There is also a problem with performance - potentially. Currenlty the list of commands available is built partially dynamically basd on configured executor - and that means that ProvidersManager and PluginsManager and discovery of avaialble providers, Hooks, Connections etc. does not have to be done when CLI methods are parsed. > > And that also means that for example `airflow --help` - and any other command - might take much longer - even 100-200 ms is a lot for those. Not sure how to address it easily though. Thanks for highlighting the potential performance issue! I will run benchmarks after adding the `CliDefinitionLoader` to measure both the overhead of adding _provider-module-level CLI definitions_ and the speedup gained by splitting out CLI definitions. -- 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]
