Bogdan Drutu
Bogdan Drutu
> I see the builder as part of the pipeline to produce the artifacts, I think it fits naturally here. I am happy to take it off if it helps...
> What are your concerns with not having maintainers? Do you think the semantic conventions repository has any downsides? I don't know if they have or not, but they don't...
> Looks like the status is still Development: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#enabled. We will need to wait for this to go stable before we can add it to the stable API. I am...
@dashpole will work on that stability, but before that can you tell me if you prefer `Enabled` (like you have in log) or `IsEnabled` (probably log needs to change).
> In such case, we may also want to update the specification as well. The choice between `IsEnabled` vs `Enabled` is allowed at SIG level since this is language canonical...
>In Trace API we have methods like IsValid, IsSampled, IsRemote. Because of this I would suggest "IsEnabled".
Duplicate of https://github.com/open-telemetry/opentelemetry-collector/issues/11405
Can you please do a PR that only adds the new proposed way (without the replacement or connection) then the second PR to start using that. As you saw there...
Trying to better understand: How do you know which element to remove?
There is a bit of a problem here, the "exporter" under the periodic reader is not a collector exporter, but an otel-go exporter and I am not sure how they...