Skip to content

kvserver: test that RangeFeed unaffected by Replica circuit breaker#76732

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
tbg:circuit-breaker-rangefeed
Feb 17, 2022
Merged

kvserver: test that RangeFeed unaffected by Replica circuit breaker#76732
craig[bot] merged 1 commit intocockroachdb:masterfrom
tbg:circuit-breaker-rangefeed

Conversation

@tbg
Copy link
Copy Markdown
Member

@tbg tbg commented Feb 17, 2022

RangeFeeds are long-running operations with a high fixed cost
(catch-up scan). They can fall behind for various reasons, replicas
becoming unavailable being one of them. However, when a replica's
circuit breaker trips, this should not abort any pending (or new, for
that matter) RangeFeeds.

Add a test that asserts that this is the case.

Touches #33007.
Touches #76146.

Release note: None

@tbg tbg requested a review from a team as a code owner February 17, 2022 13:05
@tbg tbg requested review from a team and erikgrinaker February 17, 2022 13:05
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

`RangeFeed`s are long-running operations with a high fixed cost
(catch-up scan). They can fall behind for various reasons, replicas
becoming unavailable being one of them. However, when a replica's
circuit breaker trips, this should not abort any pending (or new, for
that matter) RangeFeeds.

Add a test that asserts that this is the case.

Touches cockroachdb#33007.
Touches cockroachdb#76146.

Release note: None
@tbg tbg force-pushed the circuit-breaker-rangefeed branch from 7c7a259 to 0233176 Compare February 17, 2022 16:13
@tbg
Copy link
Copy Markdown
Member Author

tbg commented Feb 17, 2022

TFTR!

bors r=erikgrinaker

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Feb 17, 2022

Build succeeded:

@craig craig bot merged commit f2a722f into cockroachdb:master Feb 17, 2022
@tbg tbg deleted the circuit-breaker-rangefeed branch February 18, 2022 05:47
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