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] Improve async state propagation in NonSampling state #9654

Merged
merged 1 commit into from
Jan 27, 2023

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Jan 25, 2023

No description provided.

@codecov
Copy link

codecov bot commented Jan 25, 2023

Codecov Report

Merging #9654 (11ec25b) into master (6e8fdb7) will decrease coverage by 0.03%.
The diff coverage is 24.82%.

@@             Coverage Diff              @@
##             master    #9654      +/-   ##
============================================
- Coverage     38.95%   38.93%   -0.03%     
- Complexity    11904    11906       +2     
============================================
  Files          3490     3493       +3     
  Lines         93579    93675      +96     
  Branches      10501    10506       +5     
============================================
+ Hits          36458    36475      +17     
- Misses        53953    54026      +73     
- Partials       3168     3174       +6     
Impacted Files Coverage Δ
...p/pinpoint/profiler/context/AsyncDisableTrace.java 0.00% <0.00%> (ø)
...rcorp/pinpoint/profiler/context/CloseListener.java 100.00% <ø> (ø)
...t/profiler/context/DefaultAsyncContextFactory.java 39.13% <0.00%> (-1.78%) ⬇️
...p/pinpoint/profiler/context/DisableAsyncState.java 0.00% <0.00%> (ø)
...oint/profiler/context/id/ListenableAsyncState.java 71.42% <0.00%> (-3.04%) ⬇️
...profiler/context/recorder/DisableSpanRecorder.java 8.51% <0.00%> (ø)
...ler/context/recorder/WrappedSpanEventRecorder.java 26.25% <0.00%> (+0.32%) ⬆️
...ler/context/recorder/DisableSpanEventRecorder.java 12.50% <12.50%> (ø)
...oint/profiler/context/DefaultBaseTraceFactory.java 44.18% <27.27%> (-5.13%) ⬇️
...point/profiler/context/DisableAsyncChildTrace.java 30.00% <27.77%> (-2.26%) ⬇️
... and 13 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 merged commit 81fb12b into pinpoint-apm:master Jan 27, 2023
@emeroad emeroad deleted the #9595_async branch February 7, 2023 07:21
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