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

Deprecate queue retry processor and enable queueing per exporter by default #1721

Closed
pavolloffay opened this issue Sep 2, 2020 · 0 comments · Fixed by #1821
Closed

Deprecate queue retry processor and enable queueing per exporter by default #1721

pavolloffay opened this issue Sep 2, 2020 · 0 comments · Fixed by #1821
Labels
enhancement New feature or request

Comments

@pavolloffay
Copy link
Member

Is your feature request related to a problem? Please describe.

The queue retry is now defined per exporter and the processor should not be used anymore. but the docs does not say anything that the processor is deprecated. https://github.com/open-telemetry/opentelemetry-collector/tree/master/processor/queuedprocessor

The queue should be also enabled by default by exporters https://github.com/open-telemetry/opentelemetry-collector/pull/1401/files#r457838535

Describe the solution you'd like

Ad ddeprecation note to queue retry docs - readme and godoc.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context

Created from https://github.com/open-telemetry/opentelemetry-collector/pull/1401/files#r457838535
@pavolloffay pavolloffay mentioned this issue Sep 2, 2020
Merged
@tigrannajaryan tigrannajaryan added this to the GA 1.0 milestone Sep 2, 2020
@bogdandrutu bogdandrutu mentioned this issue Sep 22, 2020
Merged
@pavolloffay pavolloffay mentioned this issue Sep 22, 2020
Merged
tigrannajaryan pushed a commit that referenced this issue Sep 22, 2020
@pavolloffay
Add OTLP rec and remove qretry from example and default config (#1823)
64702fd 
Signed-off-by: Pavol Loffay <[email protected]>

Add OTLP receiver and qretry processor to example and default config.

**Link to tracking Issue:**
Updates #886 
Updates #1721 

**Testing:**
* `make run`
* build and run docker image
@andrewhsu andrewhsu added the enhancement New feature or request label Jan 6, 2021
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this issue Jul 5, 2024
@esigo
Prepare v1.7.0 release with Metrics API/SDK GA. (open-telemetry#1721)
9acde87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mark queue retry processor as deprecated. bogdandrutu/opentelemetry-collector
3 participants
@andrewhsu @tigrannajaryan @pavolloffay