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

Restore stand-alone exponential histogram mapping functions #3169

Closed
jmacd opened this issue Sep 14, 2022 · 0 comments
Closed

Restore stand-alone exponential histogram mapping functions #3169

jmacd opened this issue Sep 14, 2022 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@jmacd
Copy link
Contributor

jmacd commented Sep 14, 2022

Problem Statement

The exponential histogram mapping functions in sdk/metric/aggregator/exponential/mapping have been reviewed and are meant for stand-alone use by other OpenTelemetry components as well as for future use by the OTel metrics SDK. They were left in an orphaned directory. The directory where this lands is not critically important.

Proposed Solution

Potentially the package named sdk/metric/metricdata/histogram/exponential makes sense, with subdirectories named mapping (the already reviewed part) and structure (the same is pending review into the orphaned directory in #3022).

Alternatives

Since the data structure and mapping functions are meant to be stand-alone, they do not need to be rooted in the sdk/metric package. They could go into a stand-alone package somewhere outside of the sdk sub-package.

Additional Context

#2501 described the introduction of the stand-alone mapping functions for use by other OpenTelemetry components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants