FloChehab commented on pull request #12010: URL: https://github.com/apache/airflow/pull/12010#issuecomment-757453200
> What I would suggest follow: > > * Putting this (external) broker under `data` is not very proper to me. Putting it under `redis` is not making sense as well (this `redis` section is for the Redis instance provisioned by the Chart). So shall we have a dedicated section, say `broker`? So that it's less confusing. > > * Meanwhile, the scope of this change can be expanded from "_support external Redis_" to "_support external Broker_" (it can be Redis, RabbitMQ, or experimentally a sqlalchemy database, according to our doc). It should be straightforward because in whichever case among these three, we always deal with a URI. > > > Please share your thoughts or correct me if I missed anything. Cheers Regarding the confusion, I certainly don't have enough airflow experience to see the issue here; so I'd agree on all of that and I can quickly work on a PR for this if everyone agrees. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
