ndimiduk commented on a change in pull request #3373:
URL: https://github.com/apache/hbase/pull/3373#discussion_r651169797
##########
File path: hbase-server/src/main/resources/hbase-webapps/master/table.jsp
##########
@@ -268,7 +269,12 @@
for (int j = 0; j < numMetaReplicas; j++) {
RegionInfo meta = RegionReplicaUtil.getRegionInfoForReplica(
RegionInfoBuilder.FIRST_META_REGIONINFO, j);
- ServerName metaLocation =
MetaTableLocator.waitMetaRegionLocation(master.getZooKeeper(), j, 1);
+ ServerName metaLocation = null;
+ try {
+ metaLocation =
MetaTableLocator.waitMetaRegionLocation(master.getZooKeeper(), j, 1);
+ } catch (NotAllMetaRegionsOnlineException e) {
+ //Should ignore this Exception for we don't need to display
rit meta region info in UI
+ }
Review comment:
In this case, the user is looking at the meta table specifically. Maybe
the below `if (metaLocation != null) {...` needs an `else` block that prints a
message stating that some region location(s) could not be identified?
--
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:
[email protected]
