potiuk commented on issue #46978: URL: https://github.com/apache/airflow/issues/46978#issuecomment-2679446808
> Paging [@potiuk](https://github.com/potiuk) - you probably have thoughts or opinions on this. > > High level, sounds good. Tradeoffs moving it out of the executor and authmanager interface though. Having it there is nice in some ways, authors don't need to do conditionals on their own. 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. -- 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]
