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 377 forks source link

3.1.2 zhile #455

Closed kailll closed 3 years ago

kailll commented 3 years ago

Motivation:

bug fix

Modification:

修复采样率标识传递问题 stat 时间段整分钟打印 dataSource 部分enpoint为空 dataSource pg读写url解析 dataSource resultCode一直是N的bug

Result:

Fixes #.

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

codecov[bot] commented 3 years ago

Codecov Report

Merging #455 (cfd2d3e) into master (5245dd1) will decrease coverage by 0.07%. The diff coverage is 76.08%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #455      +/-   ##
============================================
- Coverage     46.92%   46.85%   -0.08%     
  Complexity      224      224              
============================================
  Files           331      332       +1     
  Lines          9916     9946      +30     
  Branches        943      950       +7     
============================================
+ Hits           4653     4660       +7     
- Misses         4781     4802      +21     
- Partials        482      484       +2     
Impacted Files Coverage Δ
...ource/tracer/DataSourceClientStatJsonReporter.java 94.11% <0.00%> (ø)
...atasource/tracer/DataSourceClientStatReporter.java 0.00% <0.00%> (ø)
...java/com/alipay/common/tracer/core/SofaTracer.java 71.97% <0.00%> (ø)
...er/core/configuration/SofaTracerConfiguration.java 77.46% <ø> (ø)
...at/manager/SofaTracerStatisticReporterManager.java 75.47% <60.00%> (-2.08%) :arrow_down:
...com/alipay/common/tracer/core/utils/DateUtils.java 75.00% <75.00%> (ø)
...acer/plugins/datasource/utils/DataSourceUtils.java 68.36% <81.48%> (+4.57%) :arrow_up:
...racer/core/context/span/SofaTracerSpanContext.java 79.22% <100.00%> (ø)
.../tracer/core/registry/AbstractTextB3Formatter.java 59.15% <100.00%> (ø)
...lipay/disruptor/InsufficientCapacityException.java 0.00% <0.00%> (-100.00%) :arrow_down:
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5245dd1...cfd2d3e. Read the comment docs.

xzchaoo commented 3 years ago

LGTM