Skip to content

Fix fragment commit delete ordering#5774

Merged
teo-tsirpanis merged 1 commit intoTileDB-Inc:mainfrom
cl-earthscope:cl/fix-delete-fragments-order
Mar 13, 2026
Merged

Fix fragment commit delete ordering#5774
teo-tsirpanis merged 1 commit intoTileDB-Inc:mainfrom
cl-earthscope:cl/fix-delete-fragments-order

Conversation

@cl-earthscope
Copy link
Contributor

This change reverse the delete order of fragments and their associated commits to delete the commits first.

This prevents errors that occurred previously if a commit was still present but the fragment had been removed or partially removed.

Closes #5773


TYPE: BUG
DESC: Delete commits before fragments

@cl-earthscope cl-earthscope force-pushed the cl/fix-delete-fragments-order branch from 71d1794 to 2f3a541 Compare March 12, 2026 14:53
This change reverse the delete order of fragments and their associated
commits to delete the commits first.

This prevents errors that occurred previously if a commit was still
present but the fragment had been removed or partially removed.

Closes TileDB-Inc#5773

---
TYPE: BUG
DESC: Delete commits before fragments
Copy link
Member

@teo-tsirpanis teo-tsirpanis left a comment

Choose a reason for hiding this comment

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

Thanks!

@cl-earthscope cl-earthscope force-pushed the cl/fix-delete-fragments-order branch from 2f3a541 to b02a2f6 Compare March 12, 2026 20:25
@teo-tsirpanis teo-tsirpanis merged commit c0c802f into TileDB-Inc:main Mar 13, 2026
55 of 56 checks passed
@cl-earthscope cl-earthscope deleted the cl/fix-delete-fragments-order branch March 13, 2026 16:06
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.

delete_fragments and delete_fragments_list errors due to removing the fragments before the commit

2 participants