Skip to content

mark the batchprocessor as deprecated#15046

Open
codeboten wants to merge 1 commit intoopen-telemetry:mainfrom
codeboten:codeboten/deprecate-batchprocessor
Open

mark the batchprocessor as deprecated#15046
codeboten wants to merge 1 commit intoopen-telemetry:mainfrom
codeboten:codeboten/deprecate-batchprocessor

Conversation

@codeboten
Copy link
Copy Markdown
Contributor

This continues the work to deprecate the batch processor in favour of the exporter helper's batching capabilities.

Part of #12022

This continues the work to deprecate the batch processor in favour of the exporter helper's batching capabilities.

Signed-off-by: alex boten <223565+codeboten@users.noreply.github.com>
@codeboten codeboten requested a review from a team as a code owner April 1, 2026 16:42
@codeboten codeboten requested a review from dmathieu April 1, 2026 16:42
Copy link
Copy Markdown
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - no joke, it has been coming for a while

@codeboten
Copy link
Copy Markdown
Contributor Author

LGTM - no joke, it has been coming for a while

The date it was opened on is unfortunate

Copy link
Copy Markdown
Member

@songy23 songy23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @open-telemetry/collector-approvers

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.22%. Comparing base (01b07fc) to head (36e5fb3).
⚠️ Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15046      +/-   ##
==========================================
- Coverage   91.22%   91.22%   -0.01%     
==========================================
  Files         697      697              
  Lines       44627    44629       +2     
==========================================
+ Hits        40710    40711       +1     
- Misses       2773     2775       +2     
+ Partials     1144     1143       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codeboten
Copy link
Copy Markdown
Contributor Author

note i'm not putting in the changelog to avoid merging this until we have enough approvals

Copy link
Copy Markdown
Member

@andrzej-stencel andrzej-stencel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we should deprecate it. The processor (or its v2 version if implemented) is still useful in some specific scenarios.

I propose we instead implement the fixes and also change the docs to explain that the exporter helper should be used in most scenarios, describing the use cases for this component (or rather of the new better version of it).

If we actually implement the improvements on the processor as some kind of a v2 version, then v1 could be deprecated.

@jmacd
Copy link
Copy Markdown
Contributor

jmacd commented Apr 6, 2026

I volunteered myself to write an RFC to describe the difficult transition problem we're facing here. I have not seen the proposal to support major-versioning of components referred to in #13583 (comment), but it sounds like a potential solution.

@jmacd
Copy link
Copy Markdown
Contributor

jmacd commented Apr 6, 2026

See #15047

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants