[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447954938 I mean can the JsonSurfer replace the Jayway in that class? This is an

[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447903523 Have you looked at changing the jsonpath record reader to use jsonSurfer? is there a backwards compatible way to switch to the iterator?

[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447894213 @markap14 agreed, also, this reader, as @MikeThomsen points out has a somewhat specialized use case, and cases where you wouldn't want to use

[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447890661 Maybe you can open a ticket for the Jackson upgrade This is an automated

[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447890460 @MikeThomsen what reader were you using when you had the issue? This is an

[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447889274 This also feels like we are sticking our fingers in the dyke a little. The SplitJSON processor -> has this issue The JsonPathReader ->

[GitHub] ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader.

2018-12-17 Thread GitBox
ottobackwards commented on issue #3222: NIFI-5900 Added StreamingJsonReader. URL: https://github.com/apache/nifi/pull/3222#issuecomment-447888589 So, the issue is the jsonPath, not the tree node reader. As we have discussed in NIFI-3126. I didn't do the POC on jsonSurfer because the work