Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow partial retries of trace data in queued retry processor #990

Closed
pavolloffay opened this issue May 19, 2020 · 2 comments · Fixed by #1297
Closed

Allow partial retries of trace data in queued retry processor #990

pavolloffay opened this issue May 19, 2020 · 2 comments · Fixed by #1297

Comments

@pavolloffay
Copy link
Member

The queued retry processor retries the whole batch, however some exporters (e.g. Elasticsearch bulk API) can fail only part of the request (e.g. only specific spans).

A possible non-intrusive solution could be that the returned (exporter) error would carry properties that would identify objects that should be retried.
@pavolloffay
Copy link
Member Author

cc) @flands @tigrannajaryan any thoughts?

@pavolloffay pavolloffay mentioned this issue Jun 18, 2020
Merged
3 tasks
@flands
Copy link
Contributor

flands commented Jun 23, 2020

This was discussed during the last community meeting -- it is possible but is a significant change from today. A design would be needed for this.

@pavolloffay pavolloffay mentioned this issue Jul 8, 2020
Merged
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this issue Nov 11, 2021
@MrAlias @jmacd
Release v0.10.0 (open-telemetry#990)
fd61d2e 
* Prepare for releasing v0.10.0

* Update CHANGELOG

* Correct Changelog

Co-authored-by: Joshua MacDonald <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow partial retries in queued retry processor pavolloffay/opentelemetry-collector
2 participants
@flands @pavolloffay