-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Add XXH3 Converter function to converts a value to a XXH3 hash/digest
#43032
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
Conversation
d700d87 to
5818826
Compare
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.
LGTM (checked the diff for elasticsearchexporter and signaltometricsconnector)
c55d2e4 to
1265f3c
Compare
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.
Thank you for working on this, @Frapschen! I've left a few suggestions.
|
@edmocosta Please review |
|
wire error: I didn't figure out why the |
Co-authored-by: Edmo Vamerlatti Costa <11836452+edmocosta@users.noreply.github.com>
Linter is complaining because the function |
5191514 to
62e552a
Compare
|
@edmocosta Done |
…st (open-telemetry#43032) <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description <!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. --> #### Link to tracking issue Fixes open-telemetry#42792 <!--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.--> --------- Co-authored-by: Edmo Vamerlatti Costa <11836452+edmocosta@users.noreply.github.com>
Description
Link to tracking issue
Fixes #42792
Testing
Documentation