Re: Pull request for ValidateJSON revisit

2021-09-03 Thread David Handermann
l/5326 > > Should be ready for a re-review. > > > From: Mark Payne > Sent: Friday, August 20, 2021 9:34:17 AM > To: dev@nifi.apache.org > Subject: Re: Pull request for ValidateJSON revisit > > Tim, > > It looks like I’d done a revie

Re: Pull request for ValidateJSON revisit

2021-09-03 Thread Smith, Tim
The author submitted a new pull request for ValidateJSON:https://github.com/apache/nifi/pull/5326 Should be ready for a re-review. From: Mark Payne Sent: Friday, August 20, 2021 9:34:17 AM To: dev@nifi.apache.org Subject: Re: Pull request for ValidateJSON

Re: Pull request for ValidateJSON revisit

2021-08-23 Thread Smith, Tim
Thanks Mike, Tim From: Michael Hogue Sent: Monday, August 23, 2021 8:31 AM To: dev Subject: Re: Pull request for ValidateJSON revisit Tim, The original author resubmitted the PR here: https://github.com/apache/nifi/pull/5326 Thanks for the bump, Mike On

Re: Pull request for ValidateJSON revisit

2021-08-23 Thread Michael Hogue
; > > Thanks, > > > Tim > > > > From: Mark Payne > Sent: Friday, August 20, 2021 9:34 AM > To: dev@nifi.apache.org > Subject: Re: Pull request for ValidateJSON revisit > > Tim, > > It looks like I’d done a review but then

Re: Pull request for ValidateJSON revisit

2021-08-20 Thread Smith, Tim
Mark, Will do. Since repository is gone, I'll pull in the code and resubmit a PR. Thanks, Tim From: Mark Payne Sent: Friday, August 20, 2021 9:34 AM To: dev@nifi.apache.org Subject: Re: Pull request for ValidateJSON revisit Tim, It looks like I’d

Re: Pull request for ValidateJSON revisit

2021-08-20 Thread Mark Payne
Tim, It looks like I’d done a review but then there were updates and I missed the fact that the PR had been updated. My main concern was with the licensing. It looks like I thought it was MIT but in fact it was ASL v2 (either I looked at the wrong dependency or the license was changed). Lookin

Pull request for ValidateJSON revisit

2021-08-20 Thread Smith, Tim
The pull request for a ValidateJSON processor, NIFI-7392: https://github.com/apache/nifi/pull/4232 for has been marked as stale. From the review comments, this request was near approval. There was an outstanding question on licensing that still may exist. I have a similar need for this capabil