[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3913: Adding the support for sampling logs

[GitBox <gi...@apache.org>]

mcvsubbu commented on a change in pull request #3913: Adding the support for sampling logs
URL: https://github.com/apache/incubator-pinot/pull/3913#discussion_r263044651
 
 

 ##########
 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:
   Good to seed it with some value.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org