Skip to content

Commit

Permalink
Decrease send_instructor_email_digest_tasks() batch size
Browse files Browse the repository at this point in the history
Decrease the `batch_size` of the `send_instructor_email_digest_tasks()`
Celery task.

This is because I want to try decreasing the task's `batch_size` and
correspondingly increasing its `rate_limit` (see
hypothesis/lms#5827) to see what effect that has
on performance, with an eye towards potentially removing the batching
feature altogether in order to simplify adding future features to this
code.
  • Loading branch information
seanh committed Nov 14, 2023
1 parent a634dcf commit 757a38b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion h_periodic/lms_beat.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
"send_instructor_email_digests": {
"task": "lms.tasks.email_digests.send_instructor_email_digest_tasks",
"schedule": crontab(hour=7, minute=15),
"kwargs": {"batch_size": 25},
"kwargs": {"batch_size": 10},
},
"delete_expired_task_done_rows": {
"task": "lms.tasks.task_done.delete_expired_rows",
Expand Down

0 comments on commit 757a38b

Please sign in to comment.