cadence-client icon indicating copy to clipboard operation
cadence-client copied to clipboard

Add rpc request metrics with retry in consideration

Open meiliang86 opened this issue 4 years ago • 1 comments

Is your feature request related to a problem? Please describe. Right now we have rpc metrics for single requests that does not factor in retry options. Since client does retries automatically, this does not tell the accurate story about the actual failure rate.

Proposed Solution Add metrics after all the retry attempts are exhausted. We can start with sync APIs (start/signal/signalWithStart), and maybe implement this for all if that's not messing up the code too much.

meiliang86 avatar Aug 05 '20 16:08 meiliang86

@meiliang86 I didn't understand the purposed solution. As we have backoff retry, shouldn't these retries still be counted in the failure rate? Do you mean that we should only count this as one failure after there is context timeout?

anish531213 avatar Aug 18 '20 22:08 anish531213