Bogdan Drutu

Feedback about new Metrics API, propose MustMeter

7

The new metrics API is much cleaner and more consistent with other languages, congrats on this new released API. I would like to propose a "MustMeter" as a helper/contrib: ```golang...

Confusion about duplicate register callback API and when to use one or the other

13

Right now there are 2 ways/APIs to register a callback associated with a Observer: * https://pkg.go.dev/go.opentelemetry.io/otel/[email protected]/instrument#WithInt64Callback * And RegisterCallback on https://pkg.go.dev/go.opentelemetry.io/otel/[email protected]#Meter **Questions:** * With the `RegisterCallback` you can stop collection,...

All public funcs in the config modules (e.g. `ToServer`) should accept Context.

1

- [ ] #9808 - [x] [configcompression] Update public methods to accept context.Context - [x] #9807 - [x] #9812 - [x] #9105 - [x] [configopaque] Update public methods to accept...

Define dependency hierarchy between core modules

1

We are about to release more packages as stable, and we need to have a clear structure of dependencies between packages. We need to document (maybe enforce) restrictions on which...

Remove unnecessary usages of nostd::type_traits in API.

5

The API package already uses std in other places.

ContextSlot should not be "identified" by the string name

6

https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/context/context.md#create-a-key

Remove dependency on github.com/modern-go/concurrent, sync.Map is in go 1.12

1

Signed-off-by: Bogdan Drutu