cockroach icon indicating copy to clipboard operation
cockroach copied to clipboard

kvserver: allow retrying scatter processing with more errors

Open kvoli opened this issue 9 months ago • 3 comments

Previously, scatter processing would only be retried when encountering a snapshot error. Other errors commonly occur, which we expect to be transient and retryable, such as the range descriptor changing or rejected lease transfers. The range descriptor change error being most common, due to the proclivity of clients to issue splits alongside scatter requests, which would update the range descriptor.

Retry failed scatter replicate processing if the returned error matches any of IsRetriableReplicationChangeErrors, similar to range splits. Note the maximum number of retries remains at 5 for scatter.

Resolves: #124522 Release note: None

kvoli avatar May 21 '24 22:05 kvoli