mattyb149 commented on code in PR #8362:
URL: https://github.com/apache/nifi/pull/8362#discussion_r1686893646
##########
nifi-extension-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ValidateCsv.java:
##########
@@ -120,11 +120,20 @@ public class ValidateCsv extends AbstractProcessor {
.description("The schema to be used for validation. Is expected a
comma-delimited string representing the cell "
+ "processors to apply. The following cell processors are
allowed in the schema definition: "
+ allowedOperators.toString() + ". Note: cell processors
cannot be nested except with Optional.")
- .required(true)
+ .required(false)
Review Comment:
This should have a `dependsOn(HEADER, "false")`, unless you can provide the
explicit schema yet there be a header line that should be ignored. If that's
the case maybe update the documentation for Header to reflect the current
behavior based on the different combinations of settings.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
