Skip to content

SS-320 add nonzero duration constraint to some sysvars#37344

Open
patrickwwbutler wants to merge 1 commit into
MaterializeInc:mainfrom
patrickwwbutler:patrick/ss-320
Open

SS-320 add nonzero duration constraint to some sysvars#37344
patrickwwbutler wants to merge 1 commit into
MaterializeInc:mainfrom
patrickwwbutler:patrick/ss-320

Conversation

@patrickwwbutler

Copy link
Copy Markdown
Contributor

Fixes https://linear.app/materializeinc/issue/SS-320/alter-system-set-storage-statistics-interval-0s-causes-panic-thread

and also adds the constraint to a few more interval system variables

@patrickwwbutler patrickwwbutler requested a review from a team June 29, 2026 16:04
@patrickwwbutler patrickwwbutler requested a review from a team as a code owner June 29, 2026 16:04

@martykulma martykulma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@def- def- left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For most of these parameters you could have set them to 0 system-wide, and it would be ok as long as you don't use that feature. So it's possible a customer has accidentally set them to 0. Now with this change their system would crash-loop during the upgrade, which isn't great probably.

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.

3 participants