GeorryHuang commented on a change in pull request #3374:
URL: https://github.com/apache/hbase/pull/3374#discussion_r651416613
##########
File path: hbase-server/src/main/resources/hbase-webapps/master/table.jsp
##########
@@ -378,7 +389,17 @@
for (int j = 0; j < numMetaReplicas; j++) {
RegionInfo meta = RegionReplicaUtil.getRegionInfoForReplica(
RegionInfoBuilder.FIRST_META_REGIONINFO,
j);
- ServerName metaLocation =
MetaTableLocator.waitMetaRegionLocation(master.getZooKeeper(), j, 1);
+ //If a metaLocation is null, All of its info would be empty here
to be displayed.
+ ServerName metaLocation = null;
+ try {
+ metaLocation =
MetaTableLocator.waitMetaRegionLocation(master.getZooKeeper(), j, 1);
+ } catch (NotAllMetaRegionsOnlineException e) {
+ //Since a region in transition state throw a
NotAllMetaRegionsOnlineException then
+ //could cause the whole UI crash, and the operator could have
learned that this
+ //region is in transition state from the master status UI. We
should ignore to display
+ //this region info.
+ continue;
Review comment:
After I check the current implementation it shows that If a region of
user-level table is in transition, it does not be displayed in its table
details page. So I think the meta region should be the same.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
