Skip to content

Commit

Permalink
Merge pull request #118744 from erikgrinaker/22.2-kvnemesis-barrier-a…
Browse files Browse the repository at this point in the history
…mbiguous

release-22.2: kvnemesis: ignore `Barrier` ambiguous results
  • Loading branch information
erikgrinaker authored Feb 6, 2024
2 parents 55d7e93 + 5645cfa commit ec8169c
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/kv/kvnemesis/validator.go
Original file line number Diff line number Diff line change
Expand Up @@ -439,6 +439,9 @@ func (v *validator) processOp(txnID *string, op Operation) {
// optimistically try to fit the barrier inside one of the current ranges,
// but this may race with a split, so we ignore the error in this case and
// try again later.
} else if resultIsAmbiguous(t.Result) {
// It's possible that the replica is removed by the time the barrier
// arrives. That's ok, just ignore it.
} else {
// Fail or retry on other errors, depending on type.
v.failIfError(op, t.Result)
Expand Down

0 comments on commit ec8169c

Please sign in to comment.