wu-sheng commented on pull request #7384:
URL: https://github.com/apache/skywalking/pull/7384#issuecomment-888303135


   > This patch doesn’t change the public API / script, it used qps * 60 as RPM 
internally.
   
   I know. I am afraid this could be misleading. Because `30qps * 60` is 
different with 1800rpm. In the worth case, you could get over 500qps sampled 
easily in some certain cases.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to