Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce job status page refresh rate #149

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abought
Copy link
Collaborator

@abought abought commented Apr 19, 2024

Purpose

The "job status" page provides feedback by refreshing every 5 sec.

When the queue is long, the server can experience loss of service quality just from status page checks alone. (see related connection pool issue)

The new proposed value of 20 sec will reduce server load by ~75% for running job status checks, while still giving users frequent visible feedback during "fast" parts of the process like QC.

Combined with other recent changes (DB indices, increased connection pool etc), this should be sufficient to prevent DoS when the site is used under normal conditions.

This will be included in our next fork release, and we're contributing it back for others.

Related:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant