Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/2861#discussion_r207013974 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/pom.xml --- @@ -129,6 +168,69 @@ language governing permissions and limitations under the License. --> </excludes> </configuration> </plugin> + <plugin> + <groupId>org.codehaus.mojo</groupId> --- End diff -- Do we need this section here? I thought we enabled Groovy tests from the top-level POM (although it only happens if there is also at least one Java test). If for some reason it's necessary here, I don't think you need/want to add src/main/groovy as source since the community has decided that non-test code in NiFi must be pure Java.
---