review feedback

Author: clarax

hbase-balancer/src/main/java/org/apache/hadoop/hbase/master/balancer/LoadCandidateGenerator.java

@@ -47,7 +47,8 @@ private int pickLeastLoadedServer(final BalancerClusterState cluster, int thisSe
         break;
       }
       // we don't know how many servers have the same region count, we will randomly select one
-      // using reservoir sampling (http://gregable.com/2007/10/reservoir-sampling.html)
+      // using a simplified inline reservoir sampling by assignmening a random number to  stream
+      // data and choose the greatest one. (http://gregable.com/2007/10/reservoir-sampling.html)
       double currentRandom = ThreadLocalRandom.current().nextDouble();
       if (currentRandom > currentLargestRandom) {
         selectedIndex = i;
@@ -72,7 +73,8 @@ private int pickMostLoadedServer(final BalancerClusterState cluster, int thisSer
         break;
       }
       // we don't know how many servers have the same region count, we will randomly select one
-      // using reservoir sampling (http://gregable.com/2007/10/reservoir-sampling.html)
+      // using a simplified inline reservoir sampling by assignmening a random number to  stream
+      // data and choose the greatest one. (http://gregable.com/2007/10/reservoir-sampling.html)
       double currentRandom = ThreadLocalRandom.current().nextDouble();
       if (currentRandom > currentLargestRandom) {
         selectedIndex = i;