XD-DENG commented on pull request #12010: URL: https://github.com/apache/airflow/pull/12010#issuecomment-757451877
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 ---------------------------------------------------------------- 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]
