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

add benchmark test for multi batch processing #7717

Merged

Conversation

codeboten
Copy link
Contributor

This refactors the batch processor tests to make testing multiple configuration scenarios easier. It adds a testcase for a batch processor w/ metadata keys.

@codeboten codeboten requested review from a team and bogdandrutu May 23, 2023 16:48
@codeboten codeboten added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 23, 2023
@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (2dc16f3) 91.02% compared to head (dcb847a) 91.02%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7717   +/-   ##
=======================================
  Coverage   91.02%   91.02%           
=======================================
  Files         295      295           
  Lines       14535    14535           
=======================================
  Hits        13231    13231           
  Misses       1042     1042           
  Partials      262      262           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Alex Boten added 2 commits May 23, 2023 13:57
This refactors the batch processor tests to make testing multiple configuration
scenarios easier. It adds a testcase for a batch processor w/ metadata keys.

Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
@codeboten codeboten force-pushed the codeboten/add-benchmark-metadata branch from 5dac30b to dcb847a Compare May 23, 2023 21:48
@bogdandrutu bogdandrutu merged commit 6312e62 into open-telemetry:main May 23, 2023
@github-actions github-actions bot added this to the next release milestone May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants