markap14 commented on PR #9347:
URL: https://github.com/apache/nifi/pull/9347#issuecomment-2442425790
Yeah, I think I would also agree that defaulting the `dropUnknownFields` to
`true` makes the most sense. One place where I can think of where we need to be
careful that we're doing the right thing is `ValidateRecord` but that Processor
is explicitly passing a value for the field:
```
while ((record = reader.nextRecord(coerceTypes, false)) != null) {
...
}
```
And any case in which we infer the schema, we should have an entry for every
field that is found because we infer the schema based on scanning all records
in the flowfile first.
So I think it makes sense to drop unknown fields. It also is consistent with
what the JavaDocs state will happen :) So I'm a +1 on that approach.
--
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]
