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.

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

[exporter/splunkhec] Fix logic tracking partial errors data #21720

Merged

Conversation

dmitryax
Copy link
Member

@dmitryax dmitryax commented May 10, 2023

The counters tracking the sent data did not capture the state when the buffer rejected overflown data. The issue leads to incorrect data in the partial error returned by the exporter.

@dmitryax dmitryax marked this pull request as draft May 10, 2023 00:53
@dmitryax dmitryax force-pushed the hec-exporter-remove-dead-code-path branch from af18e41 to ac19c18 Compare May 10, 2023 01:08
@dmitryax dmitryax changed the title [chore] [exporter/splunkhec] Remove dead code paths [exporter/splunkhec] Fix logic to track tracking partial errors data May 10, 2023
@dmitryax dmitryax force-pushed the hec-exporter-remove-dead-code-path branch 2 times, most recently from 50d36df to 197c530 Compare May 10, 2023 02:21
@dmitryax dmitryax marked this pull request as ready for review May 10, 2023 02:23
@dmitryax dmitryax changed the title [exporter/splunkhec] Fix logic to track tracking partial errors data [exporter/splunkhec] Fix logic tracking partial errors data May 10, 2023
@dmitryax dmitryax force-pushed the hec-exporter-remove-dead-code-path branch from 197c530 to a75f23b Compare May 10, 2023 02:26
The counters tracking the sent data did not capture the state when the buffer rejected overflown data. The issue leads to incorrect data in the partial error returned by the exporter.
@dmitryax dmitryax force-pushed the hec-exporter-remove-dead-code-path branch from a75f23b to b1b5b66 Compare May 10, 2023 02:27
@dmitryax dmitryax merged commit e160deb into open-telemetry:main May 10, 2023
@github-actions github-actions bot added this to the next release milestone May 10, 2023
@dmitryax dmitryax deleted the hec-exporter-remove-dead-code-path branch May 10, 2023 04:39
@atoulme
Copy link
Contributor

atoulme commented May 10, 2023

Thanks for the fix :)

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.

4 participants