markap14 commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447893564 I do agree that we should refactor such that we don't have so much duplication in the conversion from JsonNode to Record. I also think that we need to be careful on naming. `StreamingJsonReader` may be accurate, but the Json Tree Reader and the Json Path Reader are also both streaming readers - they stream in only a single Record at a time. We should also highlight the difference more, IMO. Perhaps provide an additionalDetails.html that discusses when to use this Reader vs. Json Path Reader or Json Tree Reader
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
