@IcySteam

✅ Deploy Preview for opentelemetry-cpp-api-docs canceled.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.06%. Comparing base (2faa400) to head (e0f2731).
Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main    #3413      +/-   ##
==========================================
- Coverage   90.06%   90.06%   -0.00%     
==========================================
  Files         212      212              
  Lines        6939     6937       -2     
==========================================
- Hits         6249     6247       -2     
  Misses        690      690              

Should we remove the metric_filter_ member from MetricCollector instead, in order to closer to match the description of the open-telemetry/opentelemetry-specification#3566?

I was not involved in the creation of the spec, but I don't think they intended to add member variables to interfaces (since that no longer would be a pure interface) . I think they just meant that child classes of MetricCollector should take in metric_filter_. Either that or as a parameter to Produce