Woutifier edited a comment on pull request #4954:
URL: https://github.com/apache/nifi/pull/4954#issuecomment-822249344


   Hi Kasonnara. The tests don't pass.
   
   This PR would change the default behavior of the CSV reader, that sounds 
(too) risky to me. Especially as trimming and processing double quotes if the 
field does not start with quotes seems at odds with RFC4180 
([https://tools.ietf.org/html/rfc4180](https://tools.ietf.org/html/rfc4180)).
   
   The best course of action if you require this behavior is to introduce a new 
property to enable/disable this setting, and keeping the default at false (i.e. 
the behavior of the CSV reader will not change if nifi users make no changes 
except for updating nifi).


-- 
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:
us...@infra.apache.org


Reply via email to