fix: Change logger scope in log appender

[gruebel]

Fixes #2747

Changes

• I was not sure what exactly is the desired state for
  

  opentelemetry-rust/opentelemetry-proto/src/transform/common.rs

  Line 57 in 1ddecb0

  let (library, target) = data;

  
  set all the values to empty strings/vec, if target is None?

Merge requirement checklist

• CONTRIBUTING guidelines followed
• Unit tests added/updated (if applicable)
• Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.6%. Comparing base (ad88615) to head (2c35ddc).

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2796   +/-   ##
=====================================
  Coverage   79.6%   79.6%           
=====================================
  Files        124     124           
  Lines      23174   23178    +4     
=====================================
+ Hits       18456   18461    +5     
+ Misses      4718    4717    -1     

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

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[cijothomas]

oh wait.. this works now?

[cijothomas]

@utpilla helped me understand better. We were unnecessarily copying the target before!

@gruebel Please do this change in a separate PR, with a before/after benchmark - it should show measurable improvement.

[cijothomas]

Fixes #2747

Changes

• I was not sure what exactly is the desired state for
  

  opentelemetry-rust/opentelemetry-proto/src/transform/common.rs

  Line 57 in 1ddecb0

  let (library, target) = data;

  set all the values to empty strings/vec, if target is None?

If target is None, then get everything from InstrumentationScope itself. If target is there, then it should be used for name/ver/schema_url. But we should still copy the Attributes from Scope. (This requires some more investigation, so we can tackle it later)