Skip to content

Conversation

xyohn
Copy link

@xyohn xyohn commented Dec 7, 2023

Motivation:

SofaTracer does not support delegating java.util.concurrent.Executor

Modification:

add com.alipay.common.tracer.core.async.SofaTraceExecutor to support delegating java.util.concurrent.Executor

Result:

Fixes #515.

If there is no issue then describe the changes introduced by this PR.

Copy link

sofastack-cla bot commented Dec 7, 2023

Hi @xyohn, welcome to SOFAStack community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

@nobodyiam
Copy link
Member

@xzchaoo @glmapper Would you please help to take a look?

Copy link

codecov bot commented Dec 21, 2023

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.85%. Comparing base (76e0574) to head (c5dd1a4).
⚠️ Report is 19 commits behind head on master.

Files with missing lines Patch % Lines
...ay/common/tracer/core/async/SofaTraceExecutor.java 0.00% 5 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #516      +/-   ##
============================================
- Coverage     56.96%   56.85%   -0.12%     
  Complexity      213      213              
============================================
  Files           226      227       +1     
  Lines          6732     6737       +5     
  Branches        687      687              
============================================
- Hits           3835     3830       -5     
- Misses         2482     2491       +9     
- Partials        415      416       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

feat: Delegate java.util.concurrent.Executor
2 participants