Skip to content

Conversation

@trask
Copy link
Member

@trask trask commented Aug 19, 2025

Resolves #7571

I did add a test, but it didn't really seem worth keeping, let me know if you want it.

I did ask Copilot if there were any other similar issues in the repo, and FWIW it came back with:

The repository appears to be well-designed with respect to casting practices. The issue #7571 was an isolated incident where one method was inconsistent with the established pattern.

@trask trask marked this pull request as ready for review August 19, 2025 22:16
@trask trask requested a review from a team as a code owner August 19, 2025 22:16
@codecov
Copy link

codecov bot commented Aug 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.99%. Comparing base (20f0a3a) to head (d71ee62).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #7597      +/-   ##
============================================
- Coverage     90.01%   89.99%   -0.02%     
+ Complexity     7081     7080       -1     
============================================
  Files           803      803              
  Lines         21419    21419              
  Branches       2086     2086              
============================================
- Hits          19280    19277       -3     
- Misses         1477     1479       +2     
- Partials        662      663       +1     

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

@jkwatson jkwatson merged commit f028e93 into open-telemetry:main Aug 20, 2025
29 checks passed
@trask trask deleted the cce branch October 20, 2025 17:27
the-clam pushed a commit to the-clam/opentelemetry-java that referenced this pull request Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MetricData.getDoubleSumData() causes ClassCastException by casting to internal ImmutableSumData

3 participants