HBASE-24369 Provide more information about merged child regions in Hb…

…ck Overlaps section, which cannot be fixed immediately (#1756) (#1769) Signed-off-by: Viraj Jasani <vjasani@apache.org> Signed-off-by: Jan Hentschel <jan.hentschel@ultratendency.com> Signed-off-by: Clara Xiong Signed-off-by: stack <stack@apache.org>
apache · May 25, 2020 · 162e1d0 · 162e1d0
1 parent c8defd4
commit 162e1d0
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 2 deletions.
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java
@@ -476,6 +476,10 @@ public List<Pair<RegionInfo, RegionInfo>> getOverlaps() {
       return this.overlaps;
     }
 
+    public Map<RegionInfo, Result> getMergedRegions() {
+      return this.mergedRegions;
+    }
+
     public List<Pair<RegionInfo, ServerName>> getUnknownServers() {
       return unknownServers;
     }

diff --git a/hbase-server/src/main/resources/hbase-webapps/master/hbck.jsp b/hbase-server/src/main/resources/hbase-webapps/master/hbck.jsp
@@ -236,6 +236,12 @@
             <div class="row inner_header">
               <div class="page-header">
                 <h2>Overlaps</h2>
+                <p>
+                  <span>
+                    Regions highlighted in <font color="blue">blue</font> are recently merged regions, HBase is still doing cleanup for them. Overlaps involving these regions cannot be fixed by <em>hbck2 fixMeta</em> at this moment.
+                    Please wait some time, run <i>catalogjanitor_run</i> in hbase shell, refresh ‘HBCK Report’ page, make sure these regions are not highlighted to start the fix.
+                  </span>
+                </p>
               </div>
             </div>
             <table class="table table-striped">
@@ -245,8 +251,16 @@
               </tr>
               <% for (Pair<RegionInfo, RegionInfo> p : report.getOverlaps()) { %>
               <tr>
-                <td><span title="<%= p.getFirst() %>"><%= p.getFirst().getEncodedName() %></span></td>
-                <td><span title="<%= p.getSecond() %>"><%= p.getSecond().getEncodedName() %></span></td>
+                <% if (report.getMergedRegions().containsKey(p.getFirst())) { %>
+                  <td><span style="color:blue;" title="<%= p.getFirst() %>"><%= p.getFirst().getEncodedName() %></span></td>
+                <% } else { %>
+                  <td><span title="<%= p.getFirst() %>"><%= p.getFirst().getEncodedName() %></span></td>
+                <% } %>
+                <% if (report.getMergedRegions().containsKey(p.getSecond())) { %>
+                  <td><span style="color:blue;" title="<%= p.getSecond() %>"><%= p.getSecond().getEncodedName() %></span></td>
+                <% } else { %>
+                  <td><span title="<%= p.getSecond() %>"><%= p.getSecond().getEncodedName() %></span></td>
+                <% } %>
               </tr>
               <% } %>