Consolidated feedback for the new Metrics SDK

MadVikingGod commented 2 years ago

The new Metrics SDK is currently in a bit of flux. It is currently being written here on this branch.

With this, it brings generics that have a number of style decisions and new types with new names. The goal of this issue is to consolidate all the feedback. If you create a PR (for a specific change) please link here to maintain visibility.

Current asks for feedback: Maintainers add and remove topics as they are discovered, and resolved.

Names: When reading through the code does the name of the struct or interface tell you what that does.
Generic style: A number of components have 3+ generics. This hurts readability and may be improved at the cost of performance.

bogdandrutu commented 2 years ago

@MadVikingGod another feedback related to no-op package for metrics https://github.com/open-telemetry/opentelemetry-go/issues/2767

MrAlias commented 2 years ago

This seems stale at this point, closing.

open-telemetry / opentelemetry-go

Consolidated feedback for the new Metrics SDK #2757