Skip to content

fix(core): fix potential storage corruption on deduplicate write resulting to same data#6360

Merged
ideoma merged 1 commit intomaster-ent-nextfrom
fix-partition-done-race-ent-next
Nov 7, 2025
Merged

fix(core): fix potential storage corruption on deduplicate write resulting to same data#6360
ideoma merged 1 commit intomaster-ent-nextfrom
fix-partition-done-race-ent-next

Conversation

@ideoma
Copy link
Copy Markdown
Collaborator

@ideoma ideoma commented Nov 7, 2025

Moves #6359 to master-ent-next

Critical storage issue introduced in #5764 and released in 9.0.0.

Very likely to happen on a materialized view refresh that results in the same data as already in the view.
The bug may result in double memory close and the deletion of the wrong directories on the box.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 7, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-partition-done-race-ent-next

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@glasstiger
Copy link
Copy Markdown
Contributor

[PR Coverage check]

😍 pass : 4 / 4 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/cairo/O3PartitionJob.java 4 4 100.00%

@ideoma ideoma merged commit 33270ba into master-ent-next Nov 7, 2025
34 checks passed
@ideoma ideoma deleted the fix-partition-done-race-ent-next branch November 7, 2025 16:43
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