[chore] Reduce complexity in processorhelper obsreport

[djaglowski]

This PR removes the recordData method, which I believe is an unhelpful abstraction for reporting internal processor metrics.

The drawbacks of the function are minor, but briefly:

1. Recording any metric requires passing in zeros for all other metrics. This is cumbersome to extend because any new metric requires updating the call to recordData for all metrics. It's also fragile because if we have more than a few metrics it is easy to position the intended metric incorrectly.
2. Every metric describes one data type, which is passed into recordData but then just used as a switch to get back to code that is specific to that data type.
3. Every call to recordData instantiates N variables, where N is the number of metrics for each data type.

All of this seems unnecessary as we can just report each metric in one unambiguous line of code.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.38%. Comparing base (a211cc8) to head (a6e9b96).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10693      +/-   ##
==========================================
- Coverage   92.39%   92.38%   -0.01%     
==========================================
  Files         403      403              
  Lines       18742    18722      -20     
==========================================
- Hits        17316    17296      -20     
  Misses       1066     1066              
  Partials      360      360              

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