hussein-awala commented on issue #32048: URL: https://github.com/apache/airflow/issues/32048#issuecomment-1601380147
I don't think we really need it, because: 1. as @potiuk mentioned above, the connections types in the webserver UI are loaded from the hooks, ensuring that the most important requirement is already met. 2. there is an Airflow configuration to enable/disable adding the default connection [database.load_default_connections](https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#config-database-load-default-connections) and in its description we explain that it's just to see how the connection looks like, and most of the users set it to `False`: > Whether to load the default connections that ship with Airflow. It’s good to get started, but you probably want to set this to False in a production environment > renaming airflow db init -> airflow db create-default-connections This command init the database if it dosen't exist, upgrade it if exist, load the default pools and synchronize `log_filename_template` with the db, why do you want to deprecate it or rename `airflow db create-default-connections`? IMO we can just change the default value of `database.load_default_connections` to `False`, and we will stop loading them by default, and the users can still load them by setting this configuration to `True`. WDYT? -- 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]
