Apache9 commented on code in PR #142: URL: https://github.com/apache/hbase-thirdparty/pull/142#discussion_r2222762171
########## pom.xml: ########## @@ -245,11 +280,7 @@ <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-compiler-plugin</artifactId> - <version>3.11.0</version> - <configuration> - <source>1.8</source> - <target>1.8</target> - </configuration> + <version>3.13.0</version> Review Comment: > yes right, just that the plugin was supposed to handle converting it to source and target, doesnot work though, just doesnot fail anymore. https://maven.apache.org/plugins/maven-compiler-plugin/examples/set-compiler-release.html#usage-on-jdk-8 OK, if it is does not fail I think the current solution is fine. We set --release in parent, and for the two modules which needs to be built with JDK8, we set source and target. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org