snleee commented on a change in pull request #3913: Adding the support for sampling logs URL: https://github.com/apache/incubator-pinot/pull/3913#discussion_r263196754
########## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java ########## @@ -65,6 +66,7 @@ public abstract class BaseBrokerRequestHandler implements BrokerRequestHandler { private static final Logger LOGGER = LoggerFactory.getLogger(BaseBrokerRequestHandler.class); private static final Pql2Compiler REQUEST_COMPILER = new Pql2Compiler(); + private static final Random RANDOM = new Random(); Review comment: changed to `maxRate` approach. This is no longer needed. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org