Robert Pająk
Robert Pająk
> OTel APIs rely on implicit context This is why in almost all places `Context` is added as required for languages requiring explicit context. I would prefer to have the...
> @pellared I can't find what you're referring to. @jmacd, I added all these in the description of https://github.com/open-telemetry/opentelemetry-specification/pull/4262 > Context is used but it refers to the OTel context...
@jsuereth > Any method which records telemetry should have it (at least optionally) included. Notice that `Enabled` is not recording telemetry. The question is whether we want to support cross-cutting...
As the author of this issue I am OK on closing (for metrics) this issue as long as the @open-telemetry/technical-committee DOES NOT block having `Context` in `Logger.Enabled`. Let's give others...
SIG meeting: We agreed to close it. The languages need to still have a possibility to add parameters to `Enabled` without breaking the users. Everyone on the call was fine...
I think it still needs programmatic support.
I think it is related to: - https://github.com/open-telemetry/opentelemetry-specification/issues/3576
Blocked by: - https://github.com/open-telemetry/opentelemetry-go/issues/5769
Unblocked
> [@open-telemetry/go-maintainers](https://github.com/orgs/open-telemetry/teams/go-maintainers) I have a question on this [#5679 (comment)](https://github.com/open-telemetry/opentelemetry-go/issues/5679#issuecomment-2774461198). I am not sure we are fully compliant. Good question. I understood it that the SDK (API implementation) should handle...