-
Notifications
You must be signed in to change notification settings - Fork 782
Closed
Description
The exponential backoff in the grpc version of the OTLPSpanExporter has its max retry timeout hard coded, meaning there is no way to configure the retry behaviour of this exporter. The value should be configurable like with the http version of the OTLPSpanExporter.
Lines 271 to 293 in 2ddb076
| def _export( | |
| self, data: Union[TypingSequence[ReadableSpan], MetricsData] | |
| ) -> ExportResultT: | |
| # After the call to shutdown, subsequent calls to Export are | |
| # not allowed and should return a Failure result. | |
| if self._shutdown: | |
| logger.warning("Exporter already shutdown, ignoring batch") | |
| return self._result.FAILURE | |
| # FIXME remove this check if the export type for traces | |
| # gets updated to a class that represents the proto | |
| # TracesData and use the code below instead. | |
| # logger.warning( | |
| # "Transient error %s encountered while exporting %s, retrying in %ss.", | |
| # error.code(), | |
| # data.__class__.__name__, | |
| # delay, | |
| # ) | |
| max_value = 64 | |
| # expo returns a generator that yields delay values which grow | |
| # exponentially. Once delay is greater than max_value, the yielded | |
| # value will remain constant. | |
| for delay in _create_exp_backoff_generator(max_value=max_value): |
andrewmwilson, awoimbee, shengyaog, krzym1 and mehtahi-deshaw
Metadata
Metadata
Assignees
Labels
No labels