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

go/storage/mkvs/checkpoint: Checkpoint in descending order #4010

Merged
merged 4 commits into from
Jun 9, 2021

Conversation

kostko
Copy link
Member

@kostko kostko commented Jun 9, 2021

Previously the checkpointer would generate checkpoints in ascending order which
meant that it could generate many checkpoints only to garbage collect them in
the next step.

@kostko kostko added the c:bug Category: bug label Jun 9, 2021
@kostko kostko force-pushed the kostko/fix/storage-checkpointer branch 2 times, most recently from 8aad760 to 890b72d Compare June 9, 2021 11:20
@kostko kostko force-pushed the kostko/fix/storage-checkpointer branch from 890b72d to 284c0b4 Compare June 9, 2021 12:41
@kostko kostko marked this pull request as ready for review June 9, 2021 12:41
Previously the checkpointer would generate checkpoints in ascending order which
meant that it could generate many checkpoints only to garbage collect them in
the next step.
@kostko kostko force-pushed the kostko/fix/storage-checkpointer branch from 284c0b4 to 642b49a Compare June 9, 2021 12:48
@kostko kostko enabled auto-merge June 9, 2021 12:53
@kostko kostko force-pushed the kostko/fix/storage-checkpointer branch from 642b49a to bb33acb Compare June 9, 2021 12:55
@kostko kostko force-pushed the kostko/fix/storage-checkpointer branch from bb33acb to d45fdf5 Compare June 9, 2021 13:04
@kostko kostko merged commit 5248989 into master Jun 9, 2021
@kostko kostko deleted the kostko/fix/storage-checkpointer branch June 9, 2021 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:bug Category: bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants