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] Refactor AsyncContext #9700

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Feb 14, 2023

@emeroad emeroad added this to the 2.5.1 milestone Feb 14, 2023
@emeroad emeroad force-pushed the #9595_nonsampling_state branch from a7adf9f to 9391a89 Compare February 14, 2023 11:11
@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Merging #9700 (27092eb) into master (2ff90ab) will decrease coverage by 0.01%.
The diff coverage is 62.50%.

@@             Coverage Diff              @@
##             master    #9700      +/-   ##
============================================
- Coverage     38.75%   38.75%   -0.01%     
+ Complexity    11921    11913       -8     
============================================
  Files          3511     3510       -1     
  Lines         94249    94258       +9     
  Branches      10589    10591       +2     
============================================
+ Hits          36523    36526       +3     
- Misses        54552    54557       +5     
- Partials       3174     3175       +1     
Impacted Files Coverage Δ
...pinpoint/profiler/context/DisableAsyncContext.java 54.83% <28.57%> (-9.17%) ⬇️
...pinpoint/profiler/context/DefaultAsyncContext.java 60.41% <37.50%> (-5.44%) ⬇️
...t/profiler/context/DefaultAsyncContextFactory.java 36.36% <50.00%> (-2.77%) ⬇️
...rcorp/pinpoint/profiler/context/AsyncContexts.java 93.33% <93.33%> (ø)
.../pinpoint/bootstrap/context/AsyncContextUtils.java 100.00% <100.00%> (ø)
...p/pinpoint/rpc/client/WriteFailFutureListener.java 56.25% <0.00%> (-25.00%) ⬇️
...vercorp/pinpoint/rpc/packet/ClientClosePacket.java 46.66% <0.00%> (-6.67%) ⬇️
...corp/pinpoint/rpc/stream/StreamChannelManager.java 56.17% <0.00%> (-5.62%) ⬇️
...ollector/cluster/zookeeper/ZookeeperJobWorker.java 78.07% <0.00%> (-1.61%) ⬇️
...orp/pinpoint/rpc/server/DefaultPinpointServer.java 79.83% <0.00%> (-1.24%) ⬇️
... and 7 more

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_nonsampling_state branch from 9391a89 to 27092eb Compare February 15, 2023 01:37
@emeroad emeroad changed the title [#9595] Refactor AsyncState [#9595] Refactor AsyncContext Feb 15, 2023
@emeroad emeroad merged commit 3702da1 into pinpoint-apm:master Feb 15, 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.

Improve async state propagation of non sampling state for full statistics
1 participant