fix: Error cleanups continued

[cijothomas]

Continuing from #2790 this actually closes #2561

Also additional cleanup done in InMemoryExporter to not use the to-be-removed-in-next-pr Error types.

[codecov]

Codecov Report

Attention: Patch coverage is 72.22222% with 5 lines in your changes missing coverage. Please review.

Project coverage is 80.9%. Comparing base (36c48db) to head (a7de7b9).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
opentelemetry-sdk/src/lib.rs	0.0%	3 Missing ⚠️
opentelemetry-otlp/src/exporter/http/mod.rs	0.0%	2 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2838   +/-   ##
=====================================
  Coverage   80.8%   80.9%           
=====================================
  Files        124     124           
  Lines      23743   23726   -17     
=====================================
  Hits       19200   19200           
+ Misses      4543    4526   -17     

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

[scottgerring]

Overall I think this is a good idea - there's not much value in propagating 1 million error variants back to a caller that ignores them - but I wonder if we shouldn't try and use Error trait impls in everything that returns errors for consistency and clarity?

[cijothomas]

Overall I think this is a good idea - there's not much value in propagating 1 million error variants back to a caller that ignores them - but I wonder if we shouldn't try and use Error trait impls in everything that returns errors for consistency and clarity?

done.