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

[QOLSVC-4633] increase number of background job workers #437

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

ThrawnCA
Copy link
Contributor

  • Adding a second worker will allow other jobs to proceed while large files are processed, at the cost of slightly increased deadlock risk

- Adding a second worker will allow other jobs to proceed while large files are processed,
at the cost of slightly increased deadlock risk
@ThrawnCA ThrawnCA requested a review from a team June 13, 2024 05:52
Copy link
Member

@duttonw duttonw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have enough ram/memory for the other workers that are also on this box?

@ThrawnCA
Copy link
Contributor Author

Do we have enough ram/memory for the other workers that are also on this box?

It's a t3a.medium, so it should handle a lot.

@duttonw
Copy link
Member

duttonw commented Aug 5, 2024

as we had occur when a 2nd batch server was running, we had major db locks occuring and just a dramatic slowdown on items. The move to split by hash may be nice but unless we have more ram for this its not that useful.

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.

2 participants