Provide default implementation for MetricExporter trait methods

[cijothomas]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 77.8%. Comparing base (d52fec7) to head (31a6bf6).
Report is 211 commits behind head on main.

Files with missing lines	Patch %	Lines
opentelemetry-sdk/src/metrics/exporter.rs	66.6%	3 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2479   +/-   ##
=====================================
  Coverage   77.8%   77.8%           
=====================================
  Files        123     124    +1     
  Lines      22864   22846   -18     
=====================================
- Hits       17800   17794    -6     
+ Misses      5064    5052   -12     

☔ 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.

[TommyCpp]

I think we should require exporter author to think through the expected behavior rather than have a blank implementation. The benifit I see is we reduce some duplication in our code base.

Also adding a default implemtation is not breaking change but removing the default implementation is. Thus I prefer to not add default implemtation. Overall I don't think the benifit is worth the risk

[cijothomas]

Closing inactive PRs. This can be discussed via an issue.