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

[#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
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