You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Mentioned here : https://github.com/open-telemetry/opentelemetry-rust/pull/1997/files#r1707954979
I wonder if should even remove SeverityText from LogRecord itself to reduce the size of LogRecord. I am not aware of any use case where user puts severity number and different text, so exporters can simply derive SeverityText from the numbered field to save perf.
The text was updated successfully, but these errors were encountered:
Agree on this. If we are always going to populate the textual representation of SeverityNumber here, there is no point of populating it in SDK. Let the exporter decide if they need it.
Mentioned here : https://github.com/open-telemetry/opentelemetry-rust/pull/1997/files#r1707954979
I wonder if should even remove SeverityText from
LogRecord
itself to reduce the size of LogRecord. I am not aware of any use case where user puts severity number and different text, so exporters can simply derive SeverityText from the numbered field to save perf.The text was updated successfully, but these errors were encountered: