Global error handler cleanup - Trace SDK

[lalitb]

Fixes #
Design discussion issue (if applicable) #

Changes

Please provide a brief description of the changes here.

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

Attention: Patch coverage is 0% with 33 lines in your changes missing coverage. Please review.

Project coverage is 79.4%. Comparing base (3742953) to head (f666a13).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
opentelemetry-sdk/src/trace/config.rs	0.0%	23 Missing ⚠️
opentelemetry-sdk/src/trace/span_processor.rs	0.0%	10 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##            main   #2256     +/-   ##
=======================================
- Coverage   79.4%   79.4%   -0.1%     
=======================================
  Files        121     121             
  Lines      20978   20981      +3     
=======================================
  Hits       16673   16673             
- Misses      4305    4308      +3     

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

[cijothomas]

Good to merge now,
Have to revisit all the logs here in the future to ensure we have the right balance between too much vs too little.

[cijothomas]

@lalitb There are quite large number of unresolved conversations in the PR, hard to tell if they are still relevant. Could you resolve the ones that are addressed and merge?

[lalitb]

@lalitb There are quite large number of unresolved conversations in the PR, hard to tell if they are still relevant. Could you resolve the ones that are addressed and merge?

I have addressed all the comments, so resolved them. Merging now. Thanks for the thorough review