simonbence commented on pull request #5530:
URL: https://github.com/apache/nifi/pull/5530#issuecomment-1014249231


   > Yep, case 1 is what I was expecting but I was surprised by the behavior of 
case 2 :)
   
   I refined the handling of the invalid (from CEF parser side) records and I 
think it is now in align with what you suggested. There is a catch however, I 
would like to highlight before closing the thread: if the invalid field is set, 
now became permanently part of the schema, which -depending on the writer- 
might result empty invalid fields in the outgoing record even for healty 
events. In your exampe, with default JsonRecordSetWriter settings this will 
happen. Of course changing the value of the "Suppress Null Values" will fix 
this, but this is something to know. Also, not every writer will be able to 
suppress this, like for example the CSV Writerr


-- 
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]


Reply via email to