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

[Snapshot V2] Integrate delete snapshot flow with pinned timestamp #15255

Closed
anshu1106 opened this issue Aug 15, 2024 · 0 comments · Fixed by #15256
Closed

[Snapshot V2] Integrate delete snapshot flow with pinned timestamp #15255

anshu1106 opened this issue Aug 15, 2024 · 0 comments · Fixed by #15256
Labels
enhancement Enhancement or improvement to existing feature or request Storage:Snapshots

Comments

@anshu1106
Copy link
Contributor

anshu1106 commented Aug 15, 2024

Is your feature request related to a problem? Please describe

  • The snapshots created with pinned timestamp need to unpin the timestamp at the time of deletion.
  • The snapshots created with pinned timestamp doesn't write per shard snapshot metadata files for which LIST call at the time of cleanup in not needed
  • The snapshots created with pinned timestamp doesn't have to explicitly cleanup lock files at the time of deletion.
    More details can be found in the RFC
  • There is a separate issue to handle remote store file cleanup for deleted indices - [Snapshot V2] Add support for remote store files cleanup #15466

Related component

Storage:Snapshots

Describe alternatives you've considered

NA

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request Storage:Snapshots
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant