Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[#11944] Include chart in /uriStat/summary response #11958

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

intr3p1d
Copy link
Contributor

No description provided.

@intr3p1d intr3p1d added this to the 3.1.0 milestone Jan 16, 2025
@intr3p1d intr3p1d self-assigned this Jan 16, 2025
Copy link

codecov bot commented Jan 16, 2025

Codecov Report

Attention: Patch coverage is 35.81395% with 138 lines in your changes missing coverage. Please review.

Project coverage is 35.02%. Comparing base (7c92d5d) to head (916a7cc).
Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
.../pinpoint/uristat/web/view/UriStatSummaryView.java 0.00% 26 Missing ⚠️
...oint/uristat/web/controller/UriStatController.java 0.00% 17 Missing ⚠️
...pinpoint/uristat/web/dao/PinotApdexSummaryDao.java 0.00% 11 Missing ⚠️
...npoint/uristat/web/dao/PinotFailureSummaryDao.java 0.00% 11 Missing ⚠️
...npoint/uristat/web/dao/PinotLatencySummaryDao.java 0.00% 11 Missing ⚠️
...pinpoint/uristat/web/dao/PinotTotalSummaryDao.java 0.00% 11 Missing ⚠️
...npoint/uristat/web/mapper/EntityToModelMapper.java 0.00% 11 Missing ⚠️
...rcorp/pinpoint/uristat/web/mapper/MapperUtils.java 47.36% 8 Missing and 2 partials ⚠️
...pinpoint/uristat/web/mapper/ModelToViewMapper.java 0.00% 5 Missing ⚠️
...int/uristat/web/chart/DefaultUriStatChartType.java 0.00% 4 Missing ⚠️
... and 11 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11958      +/-   ##
============================================
- Coverage     35.09%   35.02%   -0.08%     
+ Complexity    10981    10972       -9     
============================================
  Files          3819     3823       +4     
  Lines         91611    91768     +157     
  Branches       9699     9705       +6     
============================================
- Hits          32151    32138      -13     
- Misses        56795    56962     +167     
- Partials       2665     2668       +3     

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

@intr3p1d intr3p1d merged commit 3dc75df into pinpoint-apm:master Jan 17, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant