search

sofastack / sofa-tracer

SOFATracer is a component for the distributed system call trace. And through a unified traceId logging the logs of various network calls in the invoking link. These logs can be used for quick discovery of faults, service governance, etc.
Apache License 2.0
1.11k stars 375 forks source link

feat: Support `java.util.concurrent.Executor` （#515) #516

Open xyohn opened 10 months ago

xyohn commented 10 months ago

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.

sofastack-cla[bot] commented 10 months ago

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 commented 9 months ago

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

codecov[bot] commented 9 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (76e0574) 56.96% compared to head (c5dd1a4) 56.85%.

Files Patch % Lines
...ay/common/tracer/core/async/SofaTraceExecutor.java 0.00% 5 Missing :warning:
Additional details and impacted files ```diff @@ 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 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.