ramkrish86 commented on a change in pull request #1869:
URL: https://github.com/apache/hbase/pull/1869#discussion_r438961440



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
##########
@@ -1285,7 +1285,9 @@ private boolean waitForNamespaceOnline() throws 
InterruptedException, IOExceptio
     }
     // Else there are namespace regions up in meta. Ensure they are assigned 
before we go on.
     for (RegionInfo ri : ris) {
-      isRegionOnline(ri);
+      if (!isRegionOnline(ri)) {

Review comment:
       Same question. So even if one region is offline you will not wait for 
namespace correct? That means that indicates some case where regions are gong 
down? What if a split region was opened and that has status offline? Even then 
we wont wait for namespace?




----------------------------------------------------------------
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]


Reply via email to