potiuk commented on PR #28244: URL: https://github.com/apache/airflow/pull/28244#issuecomment-1344117907
There are quite a few other commands like that in user/role management that we could improve this way. Hey @ashb @kaxil @jhtimmins and others - do you think there is any drawback to this solution ? It does not initialize quite a lot of Airlfow Webserver pieces and we get 3x - 5x speedup for those commands. Potentially I can imagine some subtle changes that somoene might want to add in their plugins might stop working with the CLI commands if we limit initialization to only Flask App Builder. But I'd argue the CLIs of our was never meant to be extended by plugins, so I see that as "feature" not "bug". -- 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]
