NIFI-13591 Added support for multiline JSON in JSLTTransformJSON.#11141
NIFI-13591 Added support for multiline JSON in JSLTTransformJSON.#11141exceptionfactory merged 4 commits intoapache:mainfrom
Conversation
exceptionfactory
left a comment
There was a problem hiding this comment.
On initial review, I'm concerned about the Multiline Transformation Strategy property, as it makes it more difficult to reason over the Processor configuration in general. I don't think the additional property is necessary, and instead, selecting JSON Lines as the format would effectively handle each object.
|
I added the |
Thanks for clarifying, I don't think it is necessary when JSON Lines is selected. |
|
I have removed the use of the |
exceptionfactory
left a comment
There was a problem hiding this comment.
Thanks @dan-s1, the latest version looks good. +1 merging
Summary
NIFI-13591
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking
Pull Request Tracking
NIFI-00000NIFI-00000VerifiedstatusPull Request Formatting
mainbranchVerification
Please indicate the verification steps performed prior to pull request creation.
Build
./mvnw clean install -P contrib-checkLicensing
LICENSEandNOTICEfilesDocumentation