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()) {