[
https://issues.apache.org/jira/browse/AIRFLOW-4111?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
zhongjiajie closed AIRFLOW-4111.
--------------------------------
Resolution: Won't Fix
Close by XD-DENG review. reason as below:
I think it's changing the behaviour, and may cause concern (in my person
opinion): Users have been used to the behaviour that all the default
connections are somehow "fake" and they know that they need to go there
configure explicitly.
Another example: I may be using a {{read+write}} DB account for
{{sql_alchemy_conn}} ({{write}} permission must be provided), while I will use
a {{read_only}} DB account for {{airflow_db}}. But your change will use the
{{read+write}} DB account info in {{sql_alchemy_conn}} to configure
{{airflow_db}}, which should not happen by default.
IMO, the concerns this PR brings > the pros it brings (also given that {{Data
profiling/Ad Hoc Query/Known Events}} are all deprecated). Personally I would
not merge it.
Hope this clarifies.
> Auto config airflow_db to database in initdb
> --------------------------------------------
>
> Key: AIRFLOW-4111
> URL: https://issues.apache.org/jira/browse/AIRFLOW-4111
> Project: Apache Airflow
> Issue Type: Improvement
> Components: db
> Affects Versions: 1.10.2
> Reporter: zhongjiajie
> Assignee: zhongjiajie
> Priority: Minor
> Labels: features
> Fix For: 1.10.3
>
>
> Make `airflow initdb` auto configure `airflow_db` by reading and parsing
> alchemy url in `airflow.cfg` section `[core]` param `sql_alchemy_conn`.
>
> `airflow_db` work on UI `Data profiling/Ad Hoc Query` and
> `Data profiling/Known Events` but using default constant values
> which not user friendly, specially `Data profiling/Known Events`
> use conn_id `airflow_db` as example.
>
> And as we know that `airflow_db`, airflow backend database, are
> already define in `airflow.cfg` section `[core]` paratemeter
> `sql_alchemy_conn` but alchemy url type, this Jira ticket help us
> make it real.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
