You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fixes inconsistent materialized view data that may be observed on chained views with period refresh. In such scenarios the dependent view may end up with rows that are no longer present in its base view.
Also includes a breaking change. Namely, MANUAL PERIOD views are no longer automatically refreshed when a period ends. Just like with any other MANUAL view the only way to refresh a MANUAL PERIOD view now is via REFRESH MATERIALIZED VIEW SQL.
Finally, improves MatViewFuzzTest:
Improves fake clock limit calculation, so that threads do more jumps
Adds randomized sample by interval to mat view query
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 puzpuzpuz_flaky_period_mv_test
Tip
📝 Customizable high-level summaries are now available in beta!
You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Provide your own instructions using the high_level_summary_instructions setting.
Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.
Example instruction:
"Divide the high-level summary into five sections:
📝 Description — Summarize the main change in 50–60 words, explaining what was done.
📓 References — List relevant issues, discussions, documentation, or related PRs.
📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
puzpuzpuz
changed the title
test(sql): flaky MatViewFuzzTest.testSingleTablePeriodView
test(sql): add more variations to MatViewFuzzTest.testSingleTablePeriodView
Nov 27, 2025
puzpuzpuz
changed the title
test(sql): add more variations to MatViewFuzzTest.testSingleTablePeriodView
fix(sql): inconsistent state of chained period materialized view
Nov 27, 2025
puzpuzpuz
changed the title
fix(sql): inconsistent state of chained period materialized view
fix(sql): breaking change 💥 - inconsistent state of chained period materialized view
Nov 27, 2025
puzpuzpuz
changed the title
fix(sql): breaking change 💥 - inconsistent state of chained period materialized view
fix(sql): breaking change 💥 - inconsistent data in chained period materialized views
Nov 27, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes inconsistent materialized view data that may be observed on chained views with period refresh. In such scenarios the dependent view may end up with rows that are no longer present in its base view.
Also includes a breaking change. Namely, MANUAL PERIOD views are no longer automatically refreshed when a period ends. Just like with any other MANUAL view the only way to refresh a MANUAL PERIOD view now is via REFRESH MATERIALIZED VIEW SQL.
Finally, improves
MatViewFuzzTest: