You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "pan3793 (via GitHub)" <gi...@apache.org> on 2024/10/12 11:48:32 UTC

Re: [PR] HADOOP-19298. [JDK17] Add a JDK17 profile. [hadoop]

pan3793 commented on code in PR #7085:
URL: https://github.com/apache/hadoop/pull/7085#discussion_r1797688721


##########
hadoop-project/pom.xml:
##########
@@ -2737,6 +2737,46 @@
         </dependencies>
       </dependencyManagement>
     </profile>
+    <profile>
+      <id>jdk17</id>
+      <activation>
+        <jdk>[17,)</jdk>
+      </activation>
+      <properties>
+        <maven.compiler.release>17</maven.compiler.release>
+        <argLine>
+          --add-opens java.base/jdk.internal.ref=ALL-UNNAMED

Review Comment:
   @slfan1989 @adoroszlai I made a more accurate JPMS option list at https://github.com/apache/hadoop/pull/7114 by running all UTs with JDK 17, and added them to global `maven-surefire-plugin.argLine` with the option `-XX:+IgnoreUnrecognizedVMOptions`



-- 
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: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org