bito-code-review[bot] commented on code in PR #34525:
URL: https://github.com/apache/superset/pull/34525#discussion_r2772700430


##########
docs/docs/configuration/cache.mdx:
##########
@@ -80,6 +80,32 @@ instead requires a cachelib object.
 
 See [Async Queries via Celery](/docs/configuration/async-queries-celery) for 
details.
 
+## Celery beat
+
+Superset has a Celery task that will periodically warm up the cache based on 
different strategies.
+To use it, add the following to your `superset_config.py`:
+
+```python
+from celery.schedules import crontab
+
+SUPERSET_CACHE_WARMUP_USER = "user_with_permission_to_dashboards"
+
+CELERYBEAT_SCHEDULE = {

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Incorrect Celery Config Key</b></div>
   <div id="fix">
   
   The configuration uses CELERYBEAT_SCHEDULE, which is a Django Celery 
setting, but Superset's Flask-based setup uses beat_schedule in the 
CeleryConfig class. Users must update CeleryConfig.beat_schedule instead, e.g., 
by adding CeleryConfig.beat_schedule.update({...}) in superset_config.py.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #601c83</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to