Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[#11050] Refactor NodeHistogram #11118

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Jun 3, 2024

No description provided.

@emeroad emeroad added this to the 3.0.1 milestone Jun 3, 2024
Copy link

codecov bot commented Jun 3, 2024

Codecov Report

Attention: Patch coverage is 0% with 47 lines in your changes missing coverage. Please review.

Project coverage is 37.03%. Comparing base (a314e90) to head (5ddd983).

Files Patch % Lines
.../applicationmap/view/NodeHistogramSummaryView.java 0.00% 17 Missing ⚠️
...thorization/controller/ResponseTimeController.java 0.00% 11 Missing ⚠️
...t/web/applicationmap/controller/MapController.java 0.00% 6 Missing ⚠️
...orp/pinpoint/web/view/histogram/HistogramView.java 0.00% 5 Missing ⚠️
...npoint/web/view/histogram/ServerHistogramView.java 0.00% 5 Missing ⚠️
...web/applicationmap/nodes/NodeHistogramSummary.java 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #11118   +/-   ##
=========================================
  Coverage     37.03%   37.03%           
+ Complexity    11988    11987    -1     
=========================================
  Files          3948     3948           
  Lines         97290    97277   -13     
  Branches      10391    10387    -4     
=========================================
- Hits          36032    36028    -4     
+ Misses        58298    58288   -10     
- Partials       2960     2961    +1     

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

@emeroad emeroad force-pushed the #11050_NodeHistogramSummary branch from 4f2b643 to 5ddd983 Compare June 7, 2024 09:52
@emeroad emeroad merged commit 4b880d4 into pinpoint-apm:master Jun 11, 2024
2 of 3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant