Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changefeedccl: rename span-level checkpoint settings for consistency #139064

Merged

Conversation

andyyang890
Copy link
Collaborator

See individual commits


Epic: CRDB-37337

This patch renames `changefeed.frontier_checkpoint_max_bytes` to
`changefeed.span_checkpoint.max_bytes` for consistency.

Release note: None
@andyyang890 andyyang890 requested a review from asg0451 January 14, 2025 20:20
@andyyang890 andyyang890 requested a review from a team as a code owner January 14, 2025 20:20
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@andyyang890 andyyang890 force-pushed the 20250114-span-checkpoint-settings-rename branch 2 times, most recently from be9d128 to e251780 Compare January 14, 2025 20:51
@andyyang890 andyyang890 requested a review from asg0451 January 14, 2025 20:53
@andyyang890 andyyang890 force-pushed the 20250114-span-checkpoint-settings-rename branch 2 times, most recently from 049fcc8 to edfbef8 Compare January 14, 2025 21:05
This patch renames `changefeed.frontier_checkpoint_frequency` to
`changefeed.span_checkpoint.interval` for consistency.

Release note: None
…reshold setting

This patch renames
`changefeed.frontier_highwater_lag_checkpoint_threshold`
to `changefeed.span_checkpoint.lag_threshold` for consistency.

Release note (ops change): The cluster setting
`changefeed.frontier_highwater_lag_checkpoint_threshold` has been
renamed to `changefeed.span_checkpoint.lag_threshold`. The old name
remains available for backwards-compatibility.
@andyyang890 andyyang890 force-pushed the 20250114-span-checkpoint-settings-rename branch from edfbef8 to 64f47b4 Compare January 14, 2025 23:15
@andyyang890
Copy link
Collaborator Author

TFTR!

bors r=asg0451

@craig craig bot merged commit f1a746c into cockroachdb:master Jan 15, 2025
22 checks passed
@andyyang890 andyyang890 deleted the 20250114-span-checkpoint-settings-rename branch January 15, 2025 03:15
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