HBASE-22451 TestLoadIncrementalHFiles and TestSecureLoadIncrementalHFiles are flaky (#250)

Start ZKNamespaceManager as first step in TableNamespaceManager initialization

Signed-off-by: Peter Somogyi <[email protected]>

Author: apurtell

hbase-server/src/main/java/org/apache/hadoop/hbase/master/TableNamespaceManager.java

@@ -222,14 +222,16 @@ public synchronized boolean isTableAvailableAndInitialized(
       return true;
     }
 
+    if (zkNamespaceManager == null) {
+      zkNamespaceManager = new ZKNamespaceManager(masterServices.getZooKeeper());
+      zkNamespaceManager.start();
+    }
+
     // Now check if the table is assigned, if not then fail fast
     if (isTableAssigned()) {
       try {
         boolean initGoodSofar = true;
         nsTable = this.masterServices.getConnection().getTable(TableName.NAMESPACE_TABLE_NAME);
-        zkNamespaceManager = new ZKNamespaceManager(masterServices.getZooKeeper());
-        zkNamespaceManager.start();
-
         if (get(nsTable, NamespaceDescriptor.DEFAULT_NAMESPACE.getName()) == null) {
           if (createNamespaceAync) {
             masterServices.getMasterProcedureExecutor().submitProcedure(