Cijo Thomas

Given https://github.com/open-telemetry/opentelemetry-rust/pull/1738 is too close, I'd prefer to do it *after* that release.

Tagging for 0.28.0 as this must be done before stable release. This is also a pre-req for https://github.com/open-telemetry/opentelemetry-rust/issues/2386 OTLP Exporter should default to HTTP/protobuf *and* also use request::Blocking by default,...

See additional comment to be fixed while resolving this issue:https://github.com/open-telemetry/opentelemetry-rust/pull/1750#discussion_r1598799927

@lalitb Can you address for logs while fixing https://github.com/open-telemetry/opentelemetry-rust/issues/2175 ? error handler is the reason for this issue, so would be good to close them together.

@lalitb Let us know if this is ready for another review. Unfortunately, it got some conflicts too to be resolved, hopefully simple ones.

@lalitb @utpilla This is getting too big with large number of comments, making it even harder to keep up. Can you reduce scope so we can focus on one file/small-section-within-a-file...

> > @lalitb @utpilla This is getting too big with large number of comments, making it even harder to keep up. Can you reduce scope so we can focus on...

Can you describe the overall idea on how this will be implemented? This a core functionality, and there cannot be any breaking changes at all. It is intentional that the...

Closing as this is not desired. Will continue to use target to populate scope as required by the spec. Once the convention for storing attributes for appender name/version is settled,...

This may have same dependency breaking issue as https://github.com/open-telemetry/opentelemetry-rust/issues/2753#issuecomment-2704824718 If tracing-opentelemetry is not dependent on this, then we can remove it right away.