opentelemetry-rust icon indicating copy to clipboard operation
opentelemetry-rust copied to clipboard

LogRecord attributes MUST adhere to the common rules of attribute limits.

Open lalitb opened this issue 1 year ago • 1 comments

As per the specs - https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/sdk.md#logrecord-limits

lalitb avatar Nov 21 '24 02:11 lalitb

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.

cijothomas avatar Mar 10 '25 16:03 cijothomas