GitBox
2018-12-08 18:37:26 UTC
vdiravka commented on a change in pull request #1566: DRILL-6887: Fix FunctionInitializerTest.init() failure
URL: https://github.com/apache/drill/pull/1566#discussion_r240012782
##########
File path: exec/java-exec/src/test/resources/drill-udf/pom.xml
##########
@@ -28,6 +28,8 @@
<version>1.0</version>
<properties>
+ <maven.compiler.source>1.8</maven.compiler.source>
+ <maven.compiler.target>1.8</maven.compiler.target>
According to [Maven docs](https://maven.apache.org/plugins/maven-compiler-plugin/examples/set-compiler-source-and-target.html) we need to specify the properties for Java8.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
***@infra.apache.org
With regards,
Apache Git Services
URL: https://github.com/apache/drill/pull/1566#discussion_r240012782
##########
File path: exec/java-exec/src/test/resources/drill-udf/pom.xml
##########
@@ -28,6 +28,8 @@
<version>1.0</version>
<properties>
+ <maven.compiler.source>1.8</maven.compiler.source>
+ <maven.compiler.target>1.8</maven.compiler.target>
Since `maven-compiler-plugin` 3.8.0 the default value has changed from 1.5 to 1.6
Even with explicit 1.6 values for `source` and `target` properties the tests pass, but these are not the right versions for current Drill version.According to [Maven docs](https://maven.apache.org/plugins/maven-compiler-plugin/examples/set-compiler-source-and-target.html) we need to specify the properties for Java8.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
***@infra.apache.org
With regards,
Apache Git Services