pareshddevalia opened a new pull request, #679: URL: https://github.com/apache/atlas/pull/679
Changes:- Fix the python code, removed unnecessary JPMS and increase the stop wait sec updated the Readme file Refactor deprecated Mockito Matchers to ArgumentMatchers and fix failing Hive test cases Refactor Mockito imports: remove wildcard and deprecated Matchers usage Refactor Mockito imports Added support for openapi-3.0 for API documentation and fix some test-case. Bundle commons-text in hbase-hook. Fix new test-case for Repository module. What changes were proposed in this pull request? This PR introduces support for Java 17 across both build and runtime environments, while retaining compatibility with Java 8. The system can now be built and executed seamlessly using either JDK 8 or JDK 17. How was this patch tested? The patch was validated on both JDK 8 and JDK 17 through full Maven build cycles and runtime execution. Maven Build Verification JDK 8: mvn clean install JDK 17: mvn clean install Scenario 1: JDK 8 Build and Runtime Scenario 2: JDK 8 Build and JDK 17 Runtime Scenario 3: JDK 17 Build and Runtime Note for JDK 17 Users: Running integration tests (especially Jetty server-based tests) with JDK 17 requires additional JVM arguments due to stricter module access. These can be provided via MAVEN_OPTS: export MAVEN_OPTS="--add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED " Alternatively, you can place these arguments in a .mvn/jvm.config file to avoid exporting them manually for every run. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
