Github user pvillard31 commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2615#discussion_r191742124
  
    --- Diff: 
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-client-service/pom.xml
 ---
    @@ -127,8 +133,104 @@
                 <version>5.6.8</version>
                 <scope>compile</scope>
             </dependency>
    +        <dependency>
    +            <groupId>org.apache.nifi</groupId>
    +            <artifactId>nifi-avro-record-utils</artifactId>
    +            <version>1.7.0-SNAPSHOT</version>
    +            <scope>compile</scope>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.apache.nifi</groupId>
    +            <artifactId>nifi-schema-registry-service-api</artifactId>
    +            <scope>compile</scope>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.mockito</groupId>
    +            <artifactId>mockito-all</artifactId>
    +            <scope>test</scope>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.apache.nifi</groupId>
    +            <artifactId>nifi-avro-record-utils</artifactId>
    +            <version>1.7.0-SNAPSHOT</version>
    +            <scope>compile</scope>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.apache.nifi</groupId>
    +            <artifactId>nifi-avro-record-utils</artifactId>
    +            <version>1.7.0-SNAPSHOT</version>
    +            <scope>compile</scope>
    +        </dependency>
         </dependencies>
     
    +    <build>
    +        <plugins>
    +            <plugin>
    +                <groupId>org.apache.maven.plugins</groupId>
    +                <artifactId>maven-compiler-plugin</artifactId>
    +                <executions>
    +                    <!-- Only run for tests -->
    +                    <execution>
    +                        <id>groovy-tests</id>
    +                        <goals>
    +                            <goal>testCompile</goal>
    +                        </goals>
    +                        <configuration>
    +                            
<compilerId>groovy-eclipse-compiler</compilerId>
    +                        </configuration>
    +                    </execution>
    +                </executions>
    +                <configuration>
    +                    <source>1.8</source>
    --- End diff --
    
    Do we want to have this kind of configuration in low-level poms? Wondering 
if it'd be an issue with current modifications to support Java 9/10


---

Reply via email to