Skip to content

Commit

Permalink
autoscale with pool=thread crashes celery. remove and use concurrency… (
Browse files Browse the repository at this point in the history
#1929)

* autoscale with pool=thread crashes celery. remove and use concurrency instead (to be improved later)

* update dev background script as well
  • Loading branch information
rkuo-danswer authored Jul 25, 2024
1 parent 27824d6 commit 546bfbd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions backend/scripts/dev_run_background_jobs.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,8 @@ def run_jobs(exclude_indexing: bool) -> None:
"ee.danswer.background.celery.celery_app",
"worker",
"--pool=threads",
"--autoscale=3,10",
"--concurrency=16",
"--loglevel=INFO",
"--concurrency=1",
]

cmd_beat = [
Expand Down
2 changes: 1 addition & 1 deletion backend/supervisord.conf
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ autorestart=true
# relatively compute-light (e.g. they tend to just make a bunch of requests to
# Vespa / Postgres)
[program:celery_worker]
command=celery -A danswer.background.celery.celery_run:celery_app worker --pool=threads --autoscale=3,10 --loglevel=INFO --logfile=/var/log/celery_worker.log
command=celery -A danswer.background.celery.celery_run:celery_app worker --pool=threads --concurrency=16 --loglevel=INFO --logfile=/var/log/celery_worker.log
stdout_logfile=/var/log/celery_worker_supervisor.log
stdout_logfile_maxbytes=52428800
redirect_stderr=true
Expand Down

0 comments on commit 546bfbd

Please sign in to comment.