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.

Sign up for GitHub

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

Jump to bottom

Move metric SDK processor, controller, aggregator-selector packages up one level #1419

Closed
jmacd opened this issue Dec 22, 2020 · 1 comment
Closed

Move metric SDK processor, controller, aggregator-selector packages up one level #1419

jmacd opened this issue Dec 22, 2020 · 1 comment
Labels
area:metrics Part of OpenTelemetry Metrics pkg:SDK Related to an SDK package

Comments

@jmacd
Copy link
Contributor

jmacd commented Dec 22, 2020

The packages sdk/metric/processor/basic, sdk/metric/controller/basic, and sdk/metric/selector/simple
can all move up one level. This will improve readability.

See this comment: #1378 (comment)
@jmacd jmacd mentioned this issue Dec 22, 2020
Merged
@MrAlias MrAlias added area:metrics Part of OpenTelemetry Metrics pkg:SDK Related to an SDK package priority:p1 labels Jan 7, 2021
@MrAlias MrAlias added this to To do in OpenTelemetry Go RC via automation Jan 7, 2021
@punya punya removed this from To do in OpenTelemetry Go RC Feb 17, 2021
@MrAlias
Copy link
Contributor

MrAlias commented May 3, 2022

This issues relates to the current metric SDK. The metric SDK redesign project includes changes to the SDK design that will make this issue obsolete. I'm closing so no wasted work is performed to address this.

@MrAlias MrAlias closed this as completed May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:metrics Part of OpenTelemetry Metrics pkg:SDK Related to an SDK package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jmacd @MrAlias