[WIP] fix serialization/deserialization for u64 and "NaN" float fields in Metrics

[lalitb]

Fixes #2487
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)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.9%. Comparing base (1904d4b) to head (c811133).

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2491   +/-   ##
=====================================
  Coverage   77.9%   77.9%           
=====================================
  Files        123     123           
  Lines      22944   22944           
=====================================
  Hits       17879   17879           
  Misses      5065    5065           

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

[lalitb]

@nikhilsinhaparseable would it be possible for you to validate and confirm this PR against the issues mentioned in #2487 - It would be easy to get it merged in that case. Else, I will try adding unit-tests sometime next week.

[nikhilsinhaparseable]

sure @lalitb let me validate this, i will get back to you by tomorrow.

[nikhilsinhaparseable]

@lalitb histogram is fixed, I can see all the fields are getting serialized.
summary metric is still an issue, no summary record gets serialized - even after removing the quantileValues
I always get data: None for the summary metric records.

[lalitb]

summary metric is still an issue, no summary record gets serialized - even after removing the quantileValues
I always get data: None for the summary metric records.

@nikhilsinhaparseable Thanks for testing. Just to understand your use-case better - you are using opentelemetry-proto only to deserialize the data generated by otel collector? Or are you using opentelemetry-otlp as exporter to send to collector?

[nikhilsinhaparseable]

@lalitb I am using opentelemetry-proto to deserialize the data exporting from otel collector to Parseable

[nikhilsinhaparseable]

@lalitb do we have a fix for summary metric types?
currently, i have forked your PR as dependency in Parseable, but would like to know if we have any progress on the pending issue.