[skywalking] branch master updated: traceId.hashCode() Adding Math#abs (#5506)

[wusheng]

This is an automated email from the ASF dual-hosted git repository.

wusheng pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/skywalking.git


The following commit(s) were added to refs/heads/master by this push:
     new f80d851  traceId.hashCode() Adding Math#abs (#5506)
f80d851 is described below

commit f80d8519644338ffe019616ca584af08c4002e2d
Author: linliaoy <linli...@163.com>
AuthorDate: Thu Sep 17 16:38:52 2020 +0800

    traceId.hashCode() Adding Math#abs (#5506)
---
 .../analyzer/provider/trace/parser/listener/TraceSegmentSampler.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/TraceSegmentSampler.java
 
b/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/TraceSegmentSampler.java
index 493897c..4623a2c 100644
--- 
a/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/TraceSegmentSampler.java
+++ 
b/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/TraceSegmentSampler.java
@@ -32,6 +32,6 @@ public class TraceSegmentSampler {
     }
 
     public boolean shouldSample(String traceId) {
-        return traceId.hashCode() % 10000 < 
traceSampleRateWatcher.getSampleRate();
+        return Math.abs(traceId.hashCode()) % 10000 < 
traceSampleRateWatcher.getSampleRate();
     }
 }