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

Fix: Background service check daemon data pool separation and memory leak #206

Merged
merged 1 commit into from
Feb 23, 2021

Conversation

LordHepipud
Copy link
Collaborator

@LordHepipud LordHepipud commented Feb 23, 2021

Improves the background daemon by separating each single configured check into an own data pool, preventing data of leaking from one thread to another which might cause a memory leak in long term with plenty of background checks defined.

This might also reduce CPU impact because lesser data has to be processed.

@LordHepipud LordHepipud self-assigned this Feb 23, 2021
@LordHepipud LordHepipud added the Bug There is an issue present label Feb 23, 2021
@LordHepipud LordHepipud added this to the v1.4.0 milestone Feb 23, 2021
@LordHepipud LordHepipud linked an issue Feb 23, 2021 that may be closed by this pull request
@LordHepipud LordHepipud force-pushed the fix/background_service_checks_not_working branch from 74ecd7b to 461ad1f Compare February 23, 2021 09:42
@LordHepipud LordHepipud force-pushed the fix/background_service_checks_not_working branch from 461ad1f to b9eca3a Compare February 23, 2021 09:58
@LordHepipud LordHepipud merged commit ec65665 into master Feb 23, 2021
@LordHepipud LordHepipud deleted the fix/background_service_checks_not_working branch February 23, 2021 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug There is an issue present
Projects
None yet
Development

Successfully merging this pull request may close these issues.

High CPU usage
1 participant