AxelSync commented on a change in pull request #3633: NIFI-6523 MergeRecords
properties should accept expression language variables
URL: https://github.com/apache/nifi/pull/3633#discussion_r310379260
##########
File path:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/MergeRecord.java
##########
@@ -282,29 +288,89 @@ public final void resetState() {
}
if (minRecords != null && minRecords <= 0) {
results.add(new ValidationResult.Builder()
- .subject("Min Records")
- .input(String.valueOf(minRecords))
- .valid(false)
- .explanation("<Minimum Number of Records> property cannot
be negative or zero")
- .build());
+ .subject("Min Records")
Review comment:
Note: The original indentation was weird with respect to the other
validation cases
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
