elek opened a new pull request #2252: URL: https://github.com/apache/ozone/pull/2252
## What changes were proposed in this pull request? animal-sniffer maven plugin can check the API compatibility of the current code with a specific java version. It's useful when the build uses a JDK (eg. Java 8) which is different from the target JDK (eg. Java 1.6), as it can compare the used Java API signatures with the used ones. We inherited the plugin execution from the original parent Hadoop pom.xml, but it's not required as we build with Java 8 (and Java 11, but it's not important here) and our target minimum JDK is Java 8. In the meantime, it's also removed from Hadoop by https://issues.apache.org/jira/browse/HADOOP-15938, as the same functionality (if required) can be achieved by standard javac parameters (see discussion from the Hadoop Jira). On the other hand, the animal sniffer is slow. Removing it makes the build significant faster: With master: ``` mvn clean install -DskipTests -Dskip.npx -DskipShade ... [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 01:44 min [INFO] Finished at: 2021-05-17T12:40:15+02:00 [INFO] ------------------------------------------------------------------------ ``` With this patch: ``` mvn clean install -DskipTests -Dskip.npx -DskipShade ... [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 58.603 s [INFO] Finished at: 2021-05-17T12:42:05+02:00 [INFO] ------------------------------------------------------------------------ ``` It seems to be 46 / 104 -> 44% build time improvement. ## How was this patch tested? Full CI test (+ local builds) -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
