Global error handler cleanup - PeriodicReader

[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)

[lalitb]

This doesn't log actual flush error, but failure to send the result status to the channel. So keeping it debug. It would be good to also log the actual flush status as attribute here, but that would need us to clone before it is sent to channel.

[codecov]

Codecov Report

Attention: Patch coverage is 18.18182% with 9 lines in your changes missing coverage. Please review.

Project coverage is 79.6%. Comparing base (e2db7a0) to head (387c810).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
opentelemetry-sdk/src/metrics/periodic_reader.rs	18.1%	9 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2243   +/-   ##
=====================================
  Coverage   79.6%   79.6%           
=====================================
  Files        121     121           
  Lines      20918   20916    -2     
=====================================
  Hits       16666   16666           
+ Misses      4252    4250    -2     

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