You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2019/09/23 10:21:59 UTC

[GitHub] [nifi] arenger edited a comment on issue #3455: NIFI-5900 Add SelectJson processor

arenger edited a comment on issue #3455: NIFI-5900 Add SelectJson processor
URL: https://github.com/apache/nifi/pull/3455#issuecomment-534040805
 
 
   @ottobackwards Yeah I agree that this PR would probably have less maintenance burden.  That's why I closed PR #3414 a few months ago.  I would be happy to merge or to close this PR.  I'd prefer the former but just want to go one way or another.
   
   In my opinion, both this PR and PR #3222 should be merged.  This would allow for streaming json processing from either a record-based processing model or the "traditional"/original processing model.  I think #3222 and this PR complement one another, but they're not dependent on each other.  I did not submit PR #3222 and it is apparently closed.  I submitted PR #3414 but I close that one in favor of this one.  Therefore, I think this PR should be merged and @MikeThomsen can decide separately whether he would like to resubmit his work.

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


With regards,
Apache Git Services