Skip to content

clean the partition before writing to it - idempotency rule#2425

Merged
Tian-2017 merged 2 commits intomainfrom
make-sure-clean-the-partition-before-write-to-it
Sep 4, 2025
Merged

clean the partition before writing to it - idempotency rule#2425
Tian-2017 merged 2 commits intomainfrom
make-sure-clean-the-partition-before-write-to-it

Conversation

@Tian-2017
Copy link
Contributor

@Tian-2017 Tian-2017 commented Sep 3, 2025

A general rule to follow: make sure reruns always produce the same output.

I encountered an issue when re-running the Liberator copy landing to raw — it partly succeeded the first time, but the rerun caused duplicates for the entries that had already succeeded. It costs time to clean the duplications.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 3, 2025

@Tian-2017 Tian-2017 marked this pull request as ready for review September 3, 2025 09:20
@Tian-2017 Tian-2017 requested review from a team as code owners September 3, 2025 09:20
Copy link
Contributor

@LBH-wgreeff LBH-wgreeff left a comment

Choose a reason for hiding this comment

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

LGTM

@Tian-2017 Tian-2017 merged commit ef39218 into main Sep 4, 2025
9 checks passed
@Tian-2017 Tian-2017 deleted the make-sure-clean-the-partition-before-write-to-it branch September 4, 2025 14:13
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