Refactor LogExporter mod

[cijothomas]

Opening to get some early feedback.
Trying to consistently follow:
otel_sdk::signal(logs/metrics/trace)::component

so instead of opentelemetry_sdk::export::logs::{ExportResult, LogBatch, LogExporter}
we have opentelemetry_sdk::logs::{ExportResult, LogBatch, LogExporter}

(Haven't done changes to other crates. Will proceed once I get some feedback. These are breaking changes, and we need to do it before stable.)

[lalitb]

Just to be more explicit, we are discussing to have single level of namespace indirection as below:
otel_sdk::logs/metrics/trace/resource/propagator::component

+1 to the approach

[ThomsonTan]

Just to be more explicit, we are discussing to have single level of namespace indirection as below: otel_sdk::logs/metrics/trace/resource/propagator::component

+1 to the approach

Does otel_sdk::logs work? It is even shorter than opentelemetry_sdk::logs and also, it implies it is kind of short name and differentiate it from the full form of opentelemetry_logs::....

[cijothomas]

Just to be more explicit, we are discussing to have single level of namespace indirection as below: otel_sdk::logs/metrics/trace/resource/propagator::component

+1 to the approach

Yes.
Another example one I am about to send PR for is - testing::logs::InMemoryLogExporterBuilder -> logs::testing::InMemoryLogExporter

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.7%. Comparing base (b1debf0) to head (57ce632).
Report is 1 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2534   +/-   ##
=====================================
  Coverage   77.7%   77.7%           
=====================================
  Files        123     123           
  Lines      23057   23057           
=====================================
  Hits       17938   17938           
  Misses      5119    5119           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cijothomas]

Just to be more explicit, we are discussing to have single level of namespace indirection as below: otel_sdk::logs/metrics/trace/resource/propagator::component
+1 to the approach

Does otel_sdk::logs work? It is even shorter than opentelemetry_sdk::logs and also, it implies it is kind of short name and differentiate it from the full form of opentelemetry_logs::....

opentelemetry_sdk is coming from the crate name, so I don't think we can (or should) change it.

[lalitb]

Thanks for the PR.

opentelemetry_sdk::logs::log_processor_with_async_runtime will remain in multi-namespace redirection - perhaps would be difficult to fix without changing processor name.