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.

Sign up for GitHub

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

Jump to bottom

historical_uptime: regular db cleanup to optimise memory #200

Merged
merged 2 commits into from
Feb 12, 2024
Merged

historical_uptime: regular db cleanup to optimise memory #200

merged 2 commits into from
Feb 12, 2024

Conversation

bingyuyap
Copy link
Contributor

@bingyuyap bingyuyap commented Feb 7, 2024 •
edited
Loading

sequel of #199

  • added database cleanup every 48 hours to remove observations from stale messages
  • use batch update instead of updating every message to reduce transactions to db

@bingyuyap bingyuyap changed the title Bing/historical uptime db cleanup historical_uptime: db cleanup Feb 7, 2024
Base automatically changed from bing/historical-uptime-missing-signature to main February 7, 2024 19:37
@bingyuyap bingyuyap force-pushed the bing/historical-uptime-db-cleanup branch 2 times, most recently from 9beeb61 to 3a84e43 Compare February 8, 2024 00:46
@bingyuyap bingyuyap changed the title historical_uptime: db cleanup historical_uptime: regular db cleanup to optimise memory Feb 8, 2024
@bingyuyap bingyuyap force-pushed the bing/historical-uptime-db-cleanup branch from 3a84e43 to f5dfa06 Compare February 8, 2024 00:48
@bingyuyap bingyuyap marked this pull request as ready for review February 8, 2024 00:51
bruce-riley
bruce-riley reviewed Feb 8, 2024
View reviewed changes
@bingyuyap bingyuyap force-pushed the bing/historical-uptime-db-cleanup branch from f5dfa06 to 4472b34 Compare February 8, 2024 18:13
panoel
panoel reviewed Feb 8, 2024
View reviewed changes
@bingyuyap bingyuyap force-pushed the bing/historical-uptime-db-cleanup branch 2 times, most recently from d871946 to 8c49b64 Compare February 8, 2024 18:38
@bingyuyap
historical_uptime: add db cleanup to optimise memory
6f4de3b 
Signed-off-by: bingyuyap <bingyu.yap.21@gmail.com>
@bingyuyap bingyuyap force-pushed the bing/historical-uptime-db-cleanup branch from 8c49b64 to 6f4de3b Compare February 12, 2024 18:09
@bingyuyap bingyuyap merged commit 3ea71d7 into main Feb 12, 2024
3 checks passed
@bingyuyap bingyuyap deleted the bing/historical-uptime-db-cleanup branch February 12, 2024 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@panoel panoel panoel approved these changes

@evan-gray evan-gray Awaiting requested review from evan-gray

@kev1n-peters kev1n-peters Awaiting requested review from kev1n-peters

@bruce-riley bruce-riley Awaiting requested review from bruce-riley

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bingyuyap @panoel @bruce-riley