andreahlert commented on PR #61615: URL: https://github.com/apache/airflow/pull/61615#issuecomment-3866989243
Thanks, glad you like it! 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. Will regenerate the docs and address your inline comment too. -- 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]
