You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by mattyb149 <gi...@git.apache.org> on 2018/08/01 20:04:36 UTC

[GitHub] nifi pull request #2861: NIFI-5248 Added new Elasticsearch json and record p...

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.


---