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
Deprecate ValueRecorder in favor of Histogram #728
Conversation
0c930a1
to
aa9fa19
Compare
Codecov Report
@@ Coverage Diff @@
## main #728 +/- ##
=======================================
Coverage 71.50% 71.50%
=======================================
Files 102 102
Lines 8593 8593
=======================================
Hits 6144 6144
Misses 2449 2449
Continue to review full report at Codecov.
|
3cacf46
to
747739f
Compare
@TommyCpp I know the codecov are optional is it ok in this case since it's bascially just replacing a type? |
Yeah, don't worry about it especially for -0.02% changes |
747739f
to
0a91a9c
Compare
The Metric API Spec is now stable and ValueRecorder was replaced with Histogram. * Deprecations - left structs unmarked as clippy threw a fit. * Update all code examples to use Histograms. * Remove InstrumentKind::ValueRecorder since it's not part of the API. ** Otherwise we were left with duplicating code int the SDK which does exactly the same thing. Signed-off-by: Harold Dost <github@hdost.com>
0a91a9c
to
c3fc589
Compare
Only change was a rebase |
Looks good. Thanks |
The Metric API Spec is now stable and ValueRecorder was replaced with
Histogram.