HBASE-28500 Rest Java client library assumes stateless servers (Adden…

…dum:fix spotbugs warning) (#5831)
apache · Apr 17, 2024 · d7f9ba6 · d7f9ba6
1 parent 16e9aff
commit d7f9ba6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/client/Client.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/client/Client.java
@@ -35,6 +35,7 @@
 import java.util.Map;
 import java.util.Optional;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.ThreadLocalRandom;
 import javax.net.ssl.SSLContext;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.hbase.HBaseConfiguration;
@@ -270,7 +271,7 @@ public HttpResponse executePathOnly(Cluster cluster, HttpUriRequest method, Head
       throw new IOException("Cluster is empty");
     }
     if (lastNodeId == null || !sticky) {
-      lastNodeId = (int) (cluster.nodes.size() * Math.random());
+      lastNodeId = ThreadLocalRandom.current().nextInt(cluster.nodes.size());
     }
     int start = lastNodeId;
     do {