jscheffl commented on PR #61615: URL: https://github.com/apache/airflow/pull/61615#issuecomment-3866994637
> I thought about putting the URL directly in `--backend`, but it felt a bit tricky since that parameter drives other things too (docker image, test configs, CI matrix). Mixing a backend name with a raw connection string would need extra parsing to tell them apart, and we'd lose Click's enum validation for the known backends. > > Keeping `--custom-db-url` separate also maps nicely to the `AIRFLOW__DATABASE__SQL_ALCHEMY_CONN` env fallback. But if you and @potiuk prefer the single-parameter route, I can refactor - no strong feelings either way. That makes totally sense! So let's go with the current proposal in the PR. That is also OK. -- 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]
