[jira] [Comment Edited] (NIFI-5271) Move JSON Validator to a separate package and verify license/notice

["Matt Burgess (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/NIFI-5271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16502749#comment-16502749 ] 

Matt Burgess edited comment on NIFI-5271 at 6/6/18 2:17 AM:
------------------------------------------------------------

Not sure about Jackson but I think Gson would've been ok as it is ASL licensed and didn't have a NOTICE. Still, I think a separate module is a good idea, perhaps nifi-json-utils where we bring in Jackson (only because of code popularity and relative performance), so we can only include it where need be. Or perhaps BYOJ where if you depend on nifi-json-utils, you supply your own Jackson version. This is delicate, prone to runtime errors, and if not more flexible then nevermind :)


was (Author: mattyb149):
Not sure about Jackson but I think Gson would've been ok as it is ASL licensed and didn't have a NOTICE. Still, I think a separate module is a good idea, perhaps nifi-json-utils where we bring in Jackson (only because of code popularity and relative performance), so we can only include it where need be.

> Move JSON Validator to a separate package and verify license/notice
> -------------------------------------------------------------------
>
>                 Key: NIFI-5271
>                 URL: https://issues.apache.org/jira/browse/NIFI-5271
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Mike Thomsen
>            Assignee: Mike Thomsen
>            Priority: Major
>
> The JSON Validator in 5261 did not get its L&N verified and it should be moved to a separate validator module to keep Gson from being bundled where it's not needed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)