SaloniiSharmaa commented on code in PR #55344:
URL: https://github.com/apache/airflow/pull/55344#discussion_r2329234850


##########
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:
   @eladkal 
   
   Would you prefer I move the whole guide as-is, or trim it down for the 
celery_executor.rst and only add a reference to a new file? also — I’ll remove 
mentions of CeleryKubernetesExecutor. Out of curiosity, since some users might 
still be on older versions, 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?



-- 
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

Reply via email to