arenger commented on issue #3455: NIFI-5900 Add SelectJson processor
URL: https://github.com/apache/nifi/pull/3455#issuecomment-486866895
 
 
   Note that this PR is closely related to #3414.  I don't intend for both PRs 
to be accepted.  They are two different approaches to solve the same problem.  
Here are the high level questions/options that I can see:
   
   1) Create a new `SplitJsonProcessor` that uses `javax.json` (PR 3414)
   2) Create a new `SelectJson` that uses `JsonSurfer` (this PR)
   3) Keep only `SplitJson` and optionally employ a streaming approach, backed 
by `javax.json`, when a new property is set
   4) Keep only `SplitJson` and optionally employ a streaming approach, backed 
by `JsonSurfer`, when a new property is set
   
   The later two options involve possible incompatibilities between the 
underlying libraries (`javax.json` and `JsonSurfer`) and the exiting processor 
behaviors.  My vote after considering this for a while is to keep the 
`EvaluateJsonPath` processor, keep the `SplitJson` processor, and add the 
`SelectJson` processor.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to