Skip to content

Commit

Permalink
roachtest: admission-control/intent-resolution assert on 20 sublevels
Browse files Browse the repository at this point in the history
This patch addresses a lingering TODO to start asserting on 20 sublevels
since cockroachdb#109932 has been
merged.

Release note: None
  • Loading branch information
aadityasondhi committed Oct 23, 2023
1 parent 3c05cb0 commit 1bf454f
Showing 1 changed file with 1 addition and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -124,10 +124,7 @@ func registerIntentResolutionOverload(r registry.Registry) {
}
// Loop for up to 20 minutes. Intents take ~10min to resolve, and
// we're padding by another 10min.
//
// TODO(sumeer): change subLevelThreshold to 20 after intent
// resolution is subject to admission control.
const subLevelThreshold = 100
const subLevelThreshold = 20
numErrors := 0
numSuccesses := 0
latestIntentCount := math.MaxInt
Expand Down

0 comments on commit 1bf454f

Please sign in to comment.