zifeihan commented on a change in pull request #5407:
URL: https://github.com/apache/skywalking/pull/5407#discussion_r478911418
##########
File path: docs/en/setup/backend/trace-sampling.md
##########
@@ -30,4 +30,8 @@ When you set the rate different, let's say
And we assume the agents reported all trace segments to backend,
Then the 35% traces in the global will be collected and saved in storage
consistent/complete, with all spans.
20% trace segments, which reported to Backend-Instance**B**, will saved in
storage, maybe miss some trace segments,
-because they are reported to Backend-Instance**A** and ignored.
\ No newline at end of file
+because they are reported to Backend-Instance**A** and ignored.
+
+# Note
+When you open sampling, the actual sampleRate will above sampleRate. Because
we want some error segment will be saved, even that segment will abandoned by
server side trace sampling mechanism. may be miss some other trace segments,
but we can analyze this error segment to solve problem.
Review comment:
done.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
