opentelemetry-rust
opentelemetry-rust copied to clipboard
LogRecord attributes MUST adhere to the common rules of attribute limits.
As per the specs - https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/sdk.md#logrecord-limits
Spec wording is bit ambiguous:
"If the SDK implements attribute limits it MUST provide a way to change these limits"
I assumed it meant that all SDK MUST implement this feature. But the above wording uses a "If" which seem to indicate that this is optional!
I propose to remove this from Logging-sdk-stable milestone. The main enforcement is the number of attributes - which our recommended logging api, tracing, already limits to just 32. Only way one can exceed the limit is when authoring custom processors to keep adding attributes.
Unless anyone feels strongly that this must be done before stable, let's remove from the stable milestone.