[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle
r65535 commented on PR #6337: URL: https://github.com/apache/nifi/pull/6337#issuecomment-1317257062 Thanks for helping us get this over the line @exceptionfactory! -- 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. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle
r65535 commented on PR #6337: URL: https://github.com/apache/nifi/pull/6337#issuecomment-1316944568 @dan-s1 - thanks for making the changes to this PR! 拾 @exceptionfactory - do you think we could get this merged in for the 1.19.0 release? -- 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. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle
r65535 commented on PR #6337: URL: https://github.com/apache/nifi/pull/6337#issuecomment-1259586786 @exceptionfactory - do you mind taking a look at this PR? I'm keen to get it merged into the nifi code base, if possible! -- 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. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle
r65535 commented on PR #6337: URL: https://github.com/apache/nifi/pull/6337#issuecomment-1233933272 > Thanks for submitting a new pull request for this processor @r65535! > > Before reviewing the details, there appears to be a test failure in the automated build on Windows. Fixed - the JSON library supports French and English validation output, and my unit tests were validating the English content. My bad! -- 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. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org