Skip to content

perf(core): optimize mat view writing by skipping transactions that are fully replaced by future commits or full refresh (2)#6495

Merged
bluestreak01 merged 12 commits intomasterfrom
feat-optmise-mat-view-wal-apply-2
Dec 12, 2025
Merged

perf(core): optimize mat view writing by skipping transactions that are fully replaced by future commits or full refresh (2)#6495
bluestreak01 merged 12 commits intomasterfrom
feat-optmise-mat-view-wal-apply-2

Conversation

@ideoma
Copy link
Copy Markdown
Collaborator

@ideoma ideoma commented Dec 3, 2025

Restores #6416 with fix to symbol application

…re fully replaced by future commits or full refresh (2)
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 3, 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 feat-optmise-mat-view-wal-apply-2

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 : 82 / 87 (94.25%)

file detail

path covered line new line coverage
🔵 io/questdb/cairo/TableWriter.java 33 38 86.84%
🔵 io/questdb/cairo/wal/WalTxnDetails.java 3 3 100.00%
🔵 io/questdb/cairo/wal/WalWriter.java 2 2 100.00%
🔵 io/questdb/cairo/wal/ApplyWal2TableJob.java 44 44 100.00%

@questdb-butler
Copy link
Copy Markdown

⚠️ Enterprise CI Failed

The enterprise test suite failed for this PR.

Build: View Details
Tested Commit: 2d66c786e1752495cac9e32bcc46cf58500df88d

Please investigate the failure before merging.

@bluestreak01 bluestreak01 merged commit 793a859 into master Dec 12, 2025
42 of 43 checks passed
@bluestreak01 bluestreak01 deleted the feat-optmise-mat-view-wal-apply-2 branch December 12, 2025 13:42
mtopolnik pushed a commit that referenced this pull request Dec 16, 2025
…re fully replaced by future commits or full refresh (2) (#6495)
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.

4 participants