Skip to content

changefeedccl: default kv.rangefeed.catchup_scan_iterator_optimization.enabled to true in 21.2 #79727

@shermanCRL

Description

@shermanCRL

We had seen that changefeed catchup scans can be disruptive to upgrades (internal), including affecting foreground traffic, due to consuming a lot of IOPS.

I believe that we can confidently default kv.rangefeed.catchup_scan_iterator_optimization.enabled to true, which will improve the upgrade experience. See this comment.

We've had it default true on master for some time and rangefeeds have seen increased usage on master over this release cycle. Looking at the git history, I haven't seen any correctness changes to mvcc_incremental_iterator in the meantime either.

Both will be shipping by default in 22.1 and at this point we've have these settings turned on in our largest installations. So, while catchup_scan_iterator_optimization is riskier than separated_intent_scan, I would have very little hesitation in turning both on by default in 21.2 if we think it'll cut down on problems for our users.

The risk here is not zero, but is small, and I believe upgrade disruptions we’ve seen justify it. Please review this conversation.

Let’s default kv.rangefeed.catchup_scan_iterator_optimization.enabled to true in the next patch releases of v21.2.


Notes

Some care may need to be taken for mixed-versions: #73473 (comment)

PS: we'd have to make sure these new default settings respect the relevant version gates, if they don't already, to avoid breakage in mixed-version 21.1/21.2 clusters.

See also:

Jira issue: CRDB-15943

Metadata

Metadata

Assignees

Labels

A-cdcChange Data CaptureC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)E-quick-winLikely to be a quick win for someone experienced.T-cdc

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions