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

Merged
merged 1 commit into from
Jan 17, 2023

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Jan 10, 2023

No description provided.

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

codecov bot commented Jan 10, 2023

Codecov Report

Merging #9627 (7ad2a7e) into master (fdb5b0e) will decrease coverage by 0.01%.
The diff coverage is 42.71%.

@@             Coverage Diff              @@
##             master    #9627      +/-   ##
============================================
- Coverage     38.94%   38.92%   -0.02%     
  Complexity    11896    11896              
============================================
  Files          3488     3491       +3     
  Lines         93620    93642      +22     
  Branches      10501    10505       +4     
============================================
- Hits          36461    36453       -8     
- Misses        53993    54018      +25     
- Partials       3166     3171       +5     
Impacted Files Coverage Δ
...ap/interceptor/SpanRecursiveAroundInterceptor.java 0.00% <0.00%> (ø)
...user/interceptor/UserIncludeMethodInterceptor.java 0.00% <0.00%> (ø)
...ptor/ServerConnectionHandleRequestInterceptor.java 0.00% <0.00%> (ø)
...oint/profiler/context/DefaultBaseTraceFactory.java 52.27% <0.00%> (+0.58%) ⬆️
...oint/profiler/context/LoggingBaseTraceFactory.java 0.00% <0.00%> (ø)
.../profiler/context/StatefulDisableAsyncContext.java 0.00% <0.00%> (ø)
...npoint/profiler/context/id/LocalTraceRootImpl.java 81.81% <0.00%> (ø)
...point/profiler/context/id/RemoteTraceRootImpl.java 80.00% <0.00%> (ø)
.../navercorp/pinpoint/bootstrap/util/ScopeUtils.java 16.00% <20.00%> (+1.71%) ⬆️
...t/profiler/context/DefaultAsyncContextFactory.java 40.90% <22.22%> (-9.10%) ⬇️
... and 23 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_async branch 4 times, most recently from c0ad587 to b2307de Compare January 16, 2023 09:05
@emeroad emeroad merged commit e09141c into pinpoint-apm:master Jan 17, 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.

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