quickwit icon indicating copy to clipboard operation
quickwit copied to clipboard

Retry strategy for gRPC clients

Open guilload opened this issue 1 year ago • 2 comments

Exponential backoff + jitter + round robin on server pool

guilload avatar Sep 22 '22 20:09 guilload

All endpoints are not idempotent, we need to work on that too.

fmassot avatar Sep 23 '22 21:09 fmassot

Ideally I would see the retry at the tonic level. We need also to take into account that some gRPC clients use a tower balance service.

fmassot avatar Sep 30 '22 18:09 fmassot

@guilload Can you complete this ticket

fulmicoton avatar Nov 11 '22 01:11 fulmicoton

The retry work in tower is not going to land anytime soon, so we need a workaround solution in the meantime. Let's implement a RetryingMetastore that implements the usual exponential backoff with jitter retry logic. Let's replicate or reuse what we have in quickwit-aws.

guilload avatar Nov 15 '22 15:11 guilload

Closed via #2335.

guilload avatar Nov 21 '22 09:11 guilload