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

Stream DB tickets by gradually reading small batches to avoid OOM #3015

Merged
merged 5 commits into from
Oct 22, 2017

Conversation

strnadpetr
Copy link
Contributor

@strnadpetr strnadpetr commented Oct 19, 2017

When there is a large amount of tickets stored in the database, the
ticket cleanup mechanism can cause OOM when it tries to load all tickets
at once. The tickets should instead be loaded gradually in small batches

When there is a large amount of tickets stored in the database, the
ticket cleanup mechanism can cause OOM when it tries to load all tickets
at once. The tickets should instead be loaded gradually in small batches
@CLAassistant
Copy link

CLAassistant commented Oct 19, 2017

CLA assistant check
All committers have signed the CLA.

@strnadpetr
Copy link
Contributor Author

Well, now it's failing on YamlServiceRegistryDaoTests which I didn't touch

@apereo apereo deleted a comment Oct 20, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.004%) to 43.047% when pulling 987484b on strnadpetr:jpa-ticket-stream into 6737833 on apereo:master.

@mmoayyed mmoayyed merged commit 11be795 into apereo:master Oct 22, 2017
@strnadpetr strnadpetr deleted the jpa-ticket-stream branch October 23, 2017 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants