Github user mattyb149 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2246#discussion_r148575679
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteStreamCommand.java
---
@@ -222,6 +231,16 @@ public ValidationResult validate(String subject,
String input, ValidationContext
.defaultValue(";")
.build();
+ static final PropertyDescriptor PENALIZE_NONZERO_STATUS = new
PropertyDescriptor.Builder()
--- End diff --
Good point. That property is only to retain current behavior. If they leave
the defaults, then once they upgrade, their "failed" flow files will now start
to be penalized. That may be acceptable since the user will need to route or
terminate that new relationship anyway, so I could add to the doc that flow
files routed to that relationship will be penalized, which should imply that
they might need to change the Penalty Duration setting?
---
