You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by dp...@apache.org on 2023/01/13 16:56:22 UTC
[superset] branch master updated: chore(celery): update celery config variables names (#22499)
This is an automated email from the ASF dual-hosted git repository.
dpgaspar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new 31e639318b chore(celery): update celery config variables names (#22499)
31e639318b is described below
commit 31e639318b7cdb2dda17a2aa9080fa5519dee046
Author: Artem Shumeiko <53...@users.noreply.github.com>
AuthorDate: Fri Jan 13 19:56:15 2023 +0300
chore(celery): update celery config variables names (#22499)
---
CONTRIBUTING.md | 6 ++++--
docker/docker-bootstrap.sh | 2 +-
docker/pythonpath_dev/superset_config.py | 13 ++++++-------
superset/config.py | 1 -
4 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index dde9a65316..378a58eb4d 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -52,7 +52,9 @@ little bit helps, and credit will always be given.
- [Revert Guidelines](#revert-guidelines)
- [Setup Local Environment for Development](#setup-local-environment-for-development)
- [Documentation](#documentation)
- - [Images](#images)
+ - [Local Development](#local-development)
+ - [Build](#build)
+ - [Deployment](#deployment)
- [Flask server](#flask-server)
- [OS Dependencies](#os-dependencies)
- [Dependencies](#dependencies)
@@ -1293,7 +1295,7 @@ To do this, you'll need to:
- Start up a celery worker
```shell script
- celery --app=superset.tasks.celery_app:app worker -Ofair
+ celery --app=superset.tasks.celery_app:app worker -O fair
```
Note that:
diff --git a/docker/docker-bootstrap.sh b/docker/docker-bootstrap.sh
index 150f351e4b..0784a0fdf2 100755
--- a/docker/docker-bootstrap.sh
+++ b/docker/docker-bootstrap.sh
@@ -37,7 +37,7 @@ fi
if [[ "${1}" == "worker" ]]; then
echo "Starting Celery worker..."
- celery --app=superset.tasks.celery_app:app worker -Ofair -l INFO
+ celery --app=superset.tasks.celery_app:app worker -O fair -l INFO
elif [[ "${1}" == "beat" ]]; then
echo "Starting Celery beat..."
celery --app=superset.tasks.celery_app:app beat --pidfile /tmp/celerybeat.pid -l INFO -s "${SUPERSET_HOME}"/celerybeat-schedule
diff --git a/docker/pythonpath_dev/superset_config.py b/docker/pythonpath_dev/superset_config.py
index 84c1dc58ab..7bfe2012c2 100644
--- a/docker/pythonpath_dev/superset_config.py
+++ b/docker/pythonpath_dev/superset_config.py
@@ -81,13 +81,12 @@ DATA_CACHE_CONFIG = CACHE_CONFIG
class CeleryConfig(object):
- BROKER_URL = f"redis://{REDIS_HOST}:{REDIS_PORT}/{REDIS_CELERY_DB}"
- CELERY_IMPORTS = ("superset.sql_lab",)
- CELERY_RESULT_BACKEND = f"redis://{REDIS_HOST}:{REDIS_PORT}/{REDIS_RESULTS_DB}"
- CELERYD_LOG_LEVEL = "DEBUG"
- CELERYD_PREFETCH_MULTIPLIER = 1
- CELERY_ACKS_LATE = False
- CELERYBEAT_SCHEDULE = {
+ broker_url = f"redis://{REDIS_HOST}:{REDIS_PORT}/{REDIS_CELERY_DB}"
+ imports = ("superset.sql_lab",)
+ result_backend = f"redis://{REDIS_HOST}:{REDIS_PORT}/{REDIS_RESULTS_DB}"
+ worker_prefetch_multiplier = 1
+ task_acks_late = False
+ beat_schedule = {
"reports.scheduler": {
"task": "reports.scheduler",
"schedule": crontab(minute="*", hour="*"),
diff --git a/superset/config.py b/superset/config.py
index 984d1b2295..5bb495eeac 100644
--- a/superset/config.py
+++ b/superset/config.py
@@ -875,7 +875,6 @@ class CeleryConfig: # pylint: disable=too-few-public-methods
broker_url = "sqla+sqlite:///celerydb.sqlite"
imports = ("superset.sql_lab",)
result_backend = "db+sqlite:///celery_results.sqlite"
- worker_log_level = "DEBUG"
worker_prefetch_multiplier = 1
task_acks_late = False
task_annotations = {