eladkal commented on code in PR #55344: URL: https://github.com/apache/airflow/pull/55344#discussion_r2329456988
########## docs/maintenance/redis.md: ########## @@ -0,0 +1,234 @@ + + +\# Redis Maintenance + + + +\## Introduction + + + +When running Airflow with the \*\*CeleryExecutor\*\* or \*\*CeleryKubernetesExecutor\*\*, Redis is often used as the message broker. Review Comment: > do you think it’s worth adding a one-liner in the doc saying it was deprecated, or should we leave it out completely to avoid confusion? No, if we do that our docs will be impossible to maintain. When you explain about Redis that is "internal" detail of Celery. You should not be bothered if user is using `CeleryExecutor` or `CeleryKubernetesExecutor`. That is a higher level decision that is not part of what you are trying to solve. So the notion of your doc is "Assuming you are using Celery and assuming you choose Redis lets discuss how you maintains Redis" -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
