Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/2722#discussion_r189701115 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/manager/StatusMerger.java --- @@ -415,13 +415,15 @@ public static void merge(final ProcessorStatusSnapshotDTO target, final boolean target.setType(toMerge.getType()); } - // if the status to merge is invalid allow it to take precedence. whether the + // if the status to merge is validating/invalid allow it to take precedence. whether the // processor run status is disabled/stopped/running is part of the flow configuration - // and should not differ amongst nodes. however, whether a processor is invalid + // and should not differ amongst nodes. however, whether a processor is validating/invalid // can be driven by environmental conditions. this check allows any of those to // take precedence over the configured run status. - if (RunStatus.Invalid.name().equals(toMerge.getRunStatus())) { - target.setRunStatus(RunStatus.Invalid.name()); + if (RunStatus.Validating.toString().equals(toMerge.getRunStatus())) { --- End diff -- I was just addressing an inconsistency because in the `DtoFactory` we populate run status with the value from `toString()`.
---