-
Notifications
You must be signed in to change notification settings - Fork 817
Add OTLP write handler benchmark #6378
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 OTLP write handler benchmark #6378
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI doesn't run this. I am curious what are the numbers you are getting. do you mind sharing as a comment?
@friedrichg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we reuse some of the setup and timing parts of each test type to keep the code DRY?
79e6a7f
to
3073ff5
Compare
@CharlieTLe |
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
3073ff5
to
3af142e
Compare
@CharlieTLe |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
I was also able to reproduce a similar benchmark
|
This PR adds a benchmark for the OTLP write handler.
The result is as follows:
Which issue(s) this PR fixes:
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]