From b2746020bdbca75cf90c5e6d07185208ce8a74b4 Mon Sep 17 00:00:00 2001
From: chandrasekhar-188k
 <154109917+chandrasekhar-188k@users.noreply.github.com>
Date: Sat, 23 Nov 2024 22:10:33 +0530
Subject: [PATCH] HBASE-28997 Table Regions Total Locality value on the UI
 shows 1.0 even though some of the regions locality value is zero

---
 .../src/main/resources/hbase-webapps/master/table.jsp  | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/hbase-server/src/main/resources/hbase-webapps/master/table.jsp b/hbase-server/src/main/resources/hbase-webapps/master/table.jsp
index f0b8ad651171..cc3263b36164 100644
--- a/hbase-server/src/main/resources/hbase-webapps/master/table.jsp
+++ b/hbase-server/src/main/resources/hbase-webapps/master/table.jsp
@@ -32,6 +32,8 @@
   import="java.util.TreeMap"
   import="java.util.concurrent.TimeoutException"
   import="java.util.concurrent.TimeUnit"
+  import="java.text.DecimalFormat"
+  import="java.math.RoundingMode"
   import="org.apache.commons.lang3.StringEscapeUtils"
   import="org.apache.hadoop.conf.Configuration"
   import="org.apache.hadoop.hbase.HConstants"
@@ -914,10 +916,10 @@
       ((float) totalCompactedCells / totalCompactingCells)) + "%";
   }
   if (totalBlocksTotalWeight > 0) {
-    totalLocality = String.format("%.1f",
-      ((float) totalBlocksLocalWeight / totalBlocksTotalWeight));
-    totalLocalityForSsd = String.format("%.1f",
-      ((float) totalBlocksLocalWithSsdWeight / totalBlocksTotalWeight));
+    DecimalFormat df = new DecimalFormat("0.0#");
+    df.setRoundingMode(RoundingMode.DOWN);
+    totalLocality = df.format(((float) totalBlocksLocalWeight / totalBlocksTotalWeight));
+    totalLocalityForSsd = df.format(((float) totalBlocksLocalWithSsdWeight / totalBlocksTotalWeight));
   }
   if(regions != null && regions.size() > 0) { %>
 <h2>Table Regions</h2>