vvysotskyi commented on code in PR #2565:
URL: https://github.com/apache/drill/pull/2565#discussion_r883733777
##########
pom.xml:
##########
@@ -3988,13 +3988,13 @@
<jdk>[9,)</jdk>
</activation>
<properties>
+ <maven.compiler.release>8</maven.compiler.release>
<junit.args>
--add-opens=java.base/java.lang=ALL-UNNAMED
--add-opens java.base/java.net=ALL-UNNAMED
--add-opens java.base/java.nio=ALL-UNNAMED
--add-opens=java.base/java.util=ALL-UNNAMED
--add-opens java.base/sun.nio.ch=ALL-UNNAMED
- --add-opens java.security.jgss/sun.security.krb5=ALL-UNNAMED
Review Comment:
I think this line is required. There are other places where this package is
used, and build with setting maven.compiler.release to 17 will fail without it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
