Fix failing hbase-procedure tests

Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/dd7b565b
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/dd7b565b
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/dd7b565b

Branch: refs/heads/HBASE-14614
Commit: dd7b565b768e064425c653a27050b5132f7b95fb
Parents: 150d60d
Author: Michael Stack <st...@apache.org>
Authored: Sat May 6 10:45:58 2017 -0700
Committer: Michael Stack <st...@apache.org>
Committed: Tue May 23 00:33:02 2017 -0700

----------------------------------------------------------------------
 .../java/org/apache/hadoop/hbase/procedure2/Procedure.java     | 2 +-
 .../org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java  | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/dd7b565b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/Procedure.java
----------------------------------------------------------------------
diff --git 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/Procedure.java
 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/Procedure.java
index 5527076..9c47957 100644
--- 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/Procedure.java
+++ 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/Procedure.java
@@ -351,7 +351,7 @@ public abstract class Procedure<TEnvironment> implements 
Comparable<Procedure> {
    * @param builder the string builder to use to append the proc specific 
information
    */
   protected void toStringClassDetails(StringBuilder builder) {
-    builder.append(getClass().getSimpleName());
+    builder.append(getClass().getName());
   }
 
   // ==========================================================================

http://git-wip-us.apache.org/repos/asf/hbase/blob/dd7b565b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
----------------------------------------------------------------------
diff --git 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
index d36be64..e819ae8 100644
--- 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
+++ 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
@@ -1403,11 +1403,13 @@ public class ProcedureExecutor<TEnvironment> {
             reExecute = true;
             LOG.info("Short-circuit to rexecute for pid=" + 
procedure.getProcId());
           } else {
-            // yield the current procedure, and make the subprocedure runnable
+            // Yield the current procedure, and make the subprocedure runnable
+            // subprocs may come back 'null'.
             subprocs = initializeChildren(procStack, procedure, subprocs);
             LOG.info("Initialized subprocedures=" +
+              (subprocs == null? null:
                 Stream.of(subprocs).map(e -> "{" + e.toString() + "}").
-                collect(Collectors.toList()).toString());
+                collect(Collectors.toList()).toString()));
           }
         } else if (procedure.getState() == ProcedureState.WAITING_TIMEOUT) {
           if (LOG.isTraceEnabled()) {

Reply via email to