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

[#noissue] Remove isParsed state of RouteResponse #9977

Merged
merged 1 commit into from
May 22, 2023

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented May 22, 2023

No description provided.

@emeroad emeroad added this to the 2.6.0 milestone May 22, 2023
@emeroad emeroad force-pushed the #noissue_parse_state branch from 7029ddc to 58343ef Compare May 22, 2023 09:01
@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

Merging #9977 (58343ef) into master (b1572be) will decrease coverage by 0.02%.
The diff coverage is 55.88%.

@@             Coverage Diff              @@
##             master    #9977      +/-   ##
============================================
- Coverage     38.83%   38.81%   -0.02%     
- Complexity    11994    11995       +1     
============================================
  Files          3626     3627       +1     
  Lines         95132    95131       -1     
  Branches      10521    10520       -1     
============================================
- Hits          36942    36927      -15     
- Misses        55029    55043      +14     
  Partials       3161     3161              
Impacted Files Coverage Δ
...vercorp/pinpoint/web/service/AgentServiceImpl.java 0.00% <0.00%> (ø)
...corp/pinpoint/web/cluster/RouteResponseParser.java 71.42% <71.42%> (ø)
...oint/web/cluster/DefaultPinpointRouteResponse.java 44.44% <80.00%> (-13.70%) ⬇️

... and 7 files with indirect coverage changes

@emeroad emeroad merged commit 346eeea into pinpoint-apm:master May 22, 2023
@emeroad emeroad deleted the #noissue_parse_state branch May 22, 2023 10:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant