Skip to content
Permalink
Browse files
HBASE-24369 Provide more information about merged child regions in Hb…
…ck Overlaps section, which cannot be fixed immediately (#1756)

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>
  • Loading branch information
huaxiangsun committed May 25, 2020
1 parent 2382f90 commit eaa2762a4e773b973d0485a6b5654b56fe229080
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 2 deletions.
@@ -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;
}
@@ -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>
<% } %>

0 comments on commit eaa2762

Please sign in to comment.