You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by markap14 <gi...@git.apache.org> on 2018/08/29 15:14:30 UTC

[GitHub] nifi issue #2966: NIFI-5552 - Add option to normalize header column names in...

Github user markap14 commented on the issue:

    https://github.com/apache/nifi/pull/2966
  
    Hey @pvillard31 thanks for putting up a PR! I looked through it, and I do believe that it would resolve the issue that is being described in NIFI-5552. However, I'm not entirely sure that this is the best approach. This approach is adding property to the CSV Reader that are very much tailored toward Avro. I think the better approach would be to instead update the WriteAvroSchemaAttributeStrategy and AvroTypeUtil classes to normalize the names as necessary, so that Avro-related concerns are confined to the Avro-specific strategy.


---