-
Notifications
You must be signed in to change notification settings - Fork 4.1k
changefeedccl: default kv.rangefeed.catchup_scan_iterator_optimization.enabled to true in 21.2 #79727
Description
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:
- changefeedccl: make recommended cluster setttings default #76508
- changefeedccl: default kv.rangefeed.separated_intent_scan.enabled to true in v21.2 #79655
- settings: retire kv.rangefeed.separated_intent_scan.enabled #78246
- kvserver: default kv.rangefeed.catchup_scan_iterator_optimization.enabled to true #73473
Jira issue: CRDB-15943