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

YDA-5415: improve revision cleanup stability #356

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

stsnel
Copy link
Member

@stsnel stsnel commented Oct 24, 2023

This modifies the revision cleanup process, so that the scan stage (collecting data about revisions) is separate from the collect stage (collecting a list of all revisions). Scanning revision data is done in batches now, rather than a one-time scan for the whole environment.

This reduces the amount of memory needed for the scan stage, which contributes to making the revision cleanup job usable on large environments.

@stsnel stsnel force-pushed the yda-5415-revision-cleanup-scan-scalability branch 3 times, most recently from 3f38922 to 0ab086b Compare October 24, 2023 14:20
@stsnel stsnel marked this pull request as ready for review October 25, 2023 07:25
Copy link
Member

@lwesterhof lwesterhof left a comment

Choose a reason for hiding this comment

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

LGTM

revisions.py Outdated Show resolved Hide resolved
@stsnel stsnel force-pushed the yda-5415-revision-cleanup-scan-scalability branch 2 times, most recently from 154f997 to 6b75eb8 Compare October 25, 2023 08:51
This modifies the revision cleanup process, so that the scan stage
(collecting data about revisions) is separate from the collect stage
(collecting a list of all revisions). Scanning revision data is done
in batches now, rather than a one-time scan for the whole environment.

This reduces the amount of memory needed for the scan stage, which
contributes to making the revision cleanup job usable on large
environments.
@stsnel stsnel force-pushed the yda-5415-revision-cleanup-scan-scalability branch from 6b75eb8 to 7811bda Compare October 25, 2023 09:49
@stsnel stsnel merged commit 9b8cecc into development Oct 25, 2023
@stsnel stsnel deleted the yda-5415-revision-cleanup-scan-scalability branch October 25, 2023 09:57
@stsnel stsnel restored the yda-5415-revision-cleanup-scan-scalability branch October 26, 2023 11:21
@lwesterhof lwesterhof deleted the yda-5415-revision-cleanup-scan-scalability branch December 14, 2023 10:04
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