Skip to content

[exporter/splunk_hec] Deprecate 'batcher' config for now#41224

Merged
dmitryax merged 1 commit into
open-telemetry:mainfrom
dmitryax:splunk-hec-add-deprecation-warning
Jul 11, 2025
Merged

[exporter/splunk_hec] Deprecate 'batcher' config for now#41224
dmitryax merged 1 commit into
open-telemetry:mainfrom
dmitryax:splunk-hec-add-deprecation-warning

Conversation

@dmitryax

Copy link
Copy Markdown
Member

Deprecate 'batcher' config with a warning, do not remove the section yet.

@dmitryax dmitryax requested review from a team and atoulme as code owners July 10, 2025 04:09
@dmitryax dmitryax force-pushed the splunk-hec-add-deprecation-warning branch from f414d22 to 405cab9 Compare July 10, 2025 04:13
Deprecate 'batcher' config with a warning, do not remove the section  yet.
@dmitryax dmitryax force-pushed the splunk-hec-add-deprecation-warning branch from 405cab9 to 13f1e0f Compare July 10, 2025 05:01
dmitryax added a commit that referenced this pull request Jul 10, 2025
…re (#41225)

Adopt to
open-telemetry/opentelemetry-collector#13338 to
unblock the core dependency upgrade

For the end user, everything stays as is for now. Going forward, the
`batcher` section should be deprecated and removed. See
#41224
as an example. I'm leaving that part to the code owners.

---------

Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
@dmitryax dmitryax merged commit 7e05ef9 into open-telemetry:main Jul 11, 2025
177 checks passed
@github-actions github-actions Bot added this to the next release milestone Jul 11, 2025
dmitryax added a commit that referenced this pull request Jul 11, 2025
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
…re (open-telemetry#41225)

Adopt to
open-telemetry/opentelemetry-collector#13338 to
unblock the core dependency upgrade

For the end user, everything stays as is for now. Going forward, the
`batcher` section should be deprecated and removed. See
open-telemetry#41224
as an example. I'm leaving that part to the code owners.

---------

Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
…try#41224)

Deprecate 'batcher' config with a warning, do not remove the section yet.
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
songy23 added a commit that referenced this pull request Sep 25, 2025
)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
With batcher being deprecated
(#41224),
user needs to use the batch config in the `sending_queue` option.
This PR updates the readme accordingly. 
<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes

<!--Describe what testing was performed and which tests were added.-->
#### Testing

<!--Describe the documentation added.-->
#### Documentation

<!--Please delete paragraphs that you did not use before submitting.-->

---------

Signed-off-by: Dani Louca <dlouca@splunk.com>
Co-authored-by: Antoine Toulme <atoulme@splunk.com>
Co-authored-by: Curtis Robert <crobert@splunk.com>
Co-authored-by: Yang Song <songy23@users.noreply.github.com>
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.

6 participants