-
Notifications
You must be signed in to change notification settings - Fork 74
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
feat: implement WAL segments remover #680
Conversation
6b51017
to
c2dae00
Compare
c2dae00
to
359c2aa
Compare
Rebased on #705 |
3854ff5
to
2eac564
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, please squash commits.
2eac564
to
a093667
Compare
@bsbds Convert your pr to draft since CI failed |
a093667
to
d14c3c1
Compare
@bsbds Convert your pr to draft since CI failed |
d14c3c1
to
d987b8c
Compare
@bsbds Convert your pr to draft since CI failed |
d987b8c
to
2e06dc5
Compare
@Mergifyio rebase |
✅ Branch has been successfully rebased |
d714680
to
8817f47
Compare
@bsbds Convert your pr to draft since CI failed |
8817f47
to
1ed0f97
Compare
@bsbds Convert your pr to draft since CI failed |
cf46d54
to
54c90eb
Compare
@bsbds Your PR is in conflict and cannot be merged. |
8b91ae0
to
17a8c17
Compare
17a8c17
to
57860a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please squash these commits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please squash these commits
Signed-off-by: bsbds <69835502+bsbds@users.noreply.github.com>
c109919
to
26fdc28
Compare
Depends-on: #705
Please briefly answer these questions:
what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
Implement WAL segment remover, the segment remover is used to make sure a wal segment file is atomically removed.
what changes does this pull request make?
are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)