Results 83 comments of David Handermann

I'm still planning on following up on this @xuanronaldo, but recent release efforts have taken precedence. Thanks for your patience.

Thanks for the reply, and for your work on this PR @xuanronaldo! When the final version is ready, we should be able to mention your contribution through a `Co-authored-by` line...

Thank you for addressing the comments @lizhizhou. Can you remove the addition for the `QueryIoTDB` Processor? Although it also looks useful, it would be better to resolve and merge the...

Thanks for the recent updates @lizhizhou, I will take a closer look soon. At the moment, there is a merge conflict with `nifi-nar-bundles/pom.xml`, can you take a look and resolve?

@r65535 Do you have any questions on the feedback provided?

This upgrade involves some testing, as previous patch versions of Jython included some dependency differences. Tagging @mattyb149 for review.

@MikeThomsen and @SaumyaGurtu I'm not sure this PR accomplishes the goal described in NIFI-10457. Although the plugin configuration checks for duplicate dependencies, it does not highlight when certain dependencies should...

@SaumyaGurtu Although this pull request looks like a useful improvement, it does not accomplish the goals described in the associated Jira issue. The goal described in the Jira issue is...

@SaumyaGurtu, although this PR does not align with the goals of NIFI-10457, it is still a useful improvement and it does appear to implement the features described in NIFI-10588. I...

> @exceptionfactory I think you probably meant NIFI-10457 not NIFI-10467 Thanks for the correction @dan-s1!