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

Daniel Stieglitz edited comment on NIFI-12554 at 1/1/24 8:15 PM:
-----------------------------------------------------------------

[~exceptionfactory] Thanks for clarifying the intended directory structure. 
Should modules nifi-jolt-transform-json-ui  and nifi-standard-utils both in in 
nifi-nar-bundles/nifi-standard-bundle also end up under nifi-jolt-bundle? I was 
thinking the new structure would be:

* nifi-jolt-bundle
** nifi-jolt-processors
** nifi-jolt-utils (instead of nifi-standard-utils)
** nifi-jolt-transform-json-ui
** nifi-jolt-nar

Is that what you would envision?


was (Author: JIRAUSER294662):
[~exceptionfactory] Thanks for clarifying the intended directory structure. 
Should modules nifi-jolt-transform-json-ui  and nifi-standard-utils both in in 
nifi-nar-bundles/nifi-standard-bundle
also end up under nifi-jolt-bundle?

* nifi-jolt-bundle
** nifi-jolt-processors
** nifi-jolt-utils (instead of nifi-standard-utils)
** nifi-jolt-transform-json-ui
** nifi-jolt-nar

> Refactor JoltTransformJSON and JoltTransformRecord processors in order to 
> reduce duplicate code
> -----------------------------------------------------------------------------------------------
>
>                 Key: NIFI-12554
>                 URL: https://issues.apache.org/jira/browse/NIFI-12554
>             Project: Apache NiFi
>          Issue Type: Sub-task
>            Reporter: Daniel Stieglitz
>            Assignee: Daniel Stieglitz
>            Priority: Major
>
> There is a lot of duplicate code between the JoltTransformJSON and 
> JoltTransformRecord processors. As a result each time there is a bug 
> discovered in the duplicate code there has to be a fix applied in both places 
> (e.g. NIFI-11959 and NIFI-12165).  This ticket aims to pull up the common 
> code between JoltTransformJSON and JoltTransformRecord similar to what has 
> been done for PutElastisearchJSON and PutElastisearchRecord processors with 
> the creation of AbstractPutElasticsearch.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to