-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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/tanzuobservability] Refactor histogram codes #10860
Conversation
@open-telemetry/collector-approvers anyone else review this please? |
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.
Other than the changelog comment, this looks good to me from a structural level. The component's code has been validated by code owners.
CHANGELOG.md
Outdated
@@ -16,6 +16,7 @@ | |||
- `prometheusreceiver`: Add `target_info` labels to resource attributes. (#11034) | |||
- `saphanareceiver`: Fix component memory query, add better error handling (#11507) | |||
- `sapmexporter`: Add config option to log responses from Splunk APM. (#11425) | |||
- `tanzuobservabilityexporter`: Improve alorithm to translate OTEL delta exponential histograms into tanzu observability histograms (#10860) |
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.
We've recently changed how changelog entries are handled. Please, check the contribution guidelines, but in short, you'll just need to add a new file to ./unreleased.
Make histogram code be more like the WF proxy code.
Thanks @jpkrohling I added a .yaml file under the unreleased directory for CHANGELOG.md. |
Hi. Before the build was green, but since I added a .yaml file for the change log, one of the load tests is failing. Could it be a flaky test? |
Yes |
Hi @jpkrohling, I fixed the misspelling that you caught in my CHANGELOG entry. Good catch by the way. Please reach out if there is anything else. Thanks. |
…y#10860) * [exporter/tanzuobservability] Refactor histogram codes Make histogram code be more like the WF proxy code. * Get rid of redundant comments. * Enhance accuracy of delta histograms. * Switch to new method of updating CHANGELOG.md * Fix misspelling in CHANGELOG
* [exporter/tanzuobservability] Refactor histogram codes Make histogram code be more like the WF proxy code. * Get rid of redundant comments. * Enhance accuracy of delta histograms. * Switch to new method of updating CHANGELOG.md * Fix misspelling in CHANGELOG
Make histogram code be more like the WF proxy code.
Description:
Refactor the histogram code to make it more like the WF proxy code. Also make exponential delta histogram code more accurate.
Testing:
Unit testing.