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

[#9595] Fix missing close of DisableTrace #9692

Merged
merged 1 commit into from
Feb 10, 2023

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Feb 10, 2023

No description provided.

@emeroad emeroad added this to the 2.5.1 milestone Feb 10, 2023
@codecov
Copy link

codecov bot commented Feb 10, 2023

Codecov Report

Merging #9692 (edb2892) into master (d4cc508) will decrease coverage by 0.01%.
The diff coverage is 12.50%.

@@             Coverage Diff              @@
##             master    #9692      +/-   ##
============================================
- Coverage     38.75%   38.74%   -0.01%     
+ Complexity    11921    11919       -2     
============================================
  Files          3511     3511              
  Lines         94241    94246       +5     
  Branches      10590    10591       +1     
============================================
- Hits          36524    36520       -4     
- Misses        54542    54548       +6     
- Partials       3175     3178       +3     
Impacted Files Coverage Δ
...p/pinpoint/profiler/context/AsyncDisableTrace.java 0.00% <0.00%> (ø)
...ercorp/pinpoint/profiler/context/DisableTrace.java 54.00% <0.00%> (-2.25%) ⬇️
...oint/profiler/context/id/ListenableAsyncState.java 70.00% <0.00%> (-1.43%) ⬇️
...ercorp/pinpoint/profiler/context/DefaultTrace.java 58.40% <50.00%> (-0.53%) ⬇️
...vercorp/pinpoint/rpc/packet/ClientClosePacket.java 46.66% <0.00%> (-6.67%) ⬇️
...tor/metric/datasource/DefaultDataSourceMetric.java 85.18% <0.00%> (-3.71%) ⬇️
...pinpoint/profiler/monitor/DeadlockMonitorTask.java 20.45% <0.00%> (-1.14%) ⬇️
...orp/pinpoint/rpc/server/DefaultPinpointServer.java 80.65% <0.00%> (-0.42%) ⬇️
...rc/main/java/com/navercorp/pinpoint/test/Item.java 72.22% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@emeroad emeroad force-pushed the #9595_missing_close branch from c3b5a6e to edb2892 Compare February 10, 2023 07:21
@emeroad emeroad merged commit d3440f6 into pinpoint-apm:master Feb 10, 2023
# 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