Skip to content

fix(sql): overflows in materialized view refresh#6294

Merged
ideoma merged 3 commits intomaster-ent-nextfrom
puzpuzpuz_mat_view_refresh_step_overflow2
Oct 22, 2025
Merged

fix(sql): overflows in materialized view refresh#6294
ideoma merged 3 commits intomaster-ent-nextfrom
puzpuzpuz_mat_view_refresh_step_overflow2

Conversation

@puzpuzpuz
Copy link
Copy Markdown
Contributor

Port of #6233 and #6258

@puzpuzpuz puzpuzpuz self-assigned this Oct 22, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 22, 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 puzpuzpuz_mat_view_refresh_step_overflow2

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 : 25 / 28 (89.29%)

file detail

path covered line new line coverage
🔵 io/questdb/cairo/DefaultCairoConfiguration.java 3 4 75.00%
🔵 io/questdb/cairo/mv/MatViewRefreshJob.java 15 17 88.24%
🔵 io/questdb/PropServerConfiguration.java 5 5 100.00%
🔵 io/questdb/PropertyKey.java 1 1 100.00%
🔵 io/questdb/cairo/CairoConfigurationWrapper.java 1 1 100.00%

@ideoma ideoma merged commit aee3ae5 into master-ent-next Oct 22, 2025
35 checks passed
@ideoma ideoma deleted the puzpuzpuz_mat_view_refresh_step_overflow2 branch October 22, 2025 13:39
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