Skip to content

Draft: `GrpcRetry`: return number of retries

Santiago Gil requested to merge santigl/grpc-retry-count into master

Before raising this MR, consider whether the following are required, and complete if so:

  • Unit tests
  • [ ] Metrics
  • [ ] Documentation update(s)

If not required, please explain in brief why not.

Description

This makes the GrpcRetry::retry() expose to the caller the number of retries that were performed after the initial request.

Changes proposed in this merge request:

  • GrpcError: Add numRetries member to store the number of retries before retry() threw.
  • GrpcRetry::retry(): return number of retries (excluding the initial request) performed for a gRPC invocation. In the successful case by value; otherwise attached to the thrown exception.

Issues addressed

Closes #60 (closed)

Edited by Santiago Gil

Merge request reports