LuciferYang opened a new pull request #34376:
URL: https://github.com/apache/spark/pull/34376


   ### What changes were proposed in this pull request?
   The main change of this pr as follows:
   
   - Add `extraJavaTestArgs` to `scalatest-maven-plugin` in `sql/hive/pom.xml`
   - Add  `--add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED` to 
default java module options because some UTs failed due to
   
   ```
   Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make 
field private volatile int java.util.concurrent.atomic.AtomicBoolean.value 
accessible: module java.base does not "opens java.util.concurrent.atomic" to 
unnamed module @2744cc4a
   ```
    
   
   ### Why are the changes needed?
   Pass UT with JDK 17
   
   
   
   
   ### Does this PR introduce _any_ user-facing change?
   No.
   
   
   ### How was this patch tested?
   
   - Pass the Jenkins or GitHub Action
   - Manual test use Java17 build + Java17 run (both openjdk version "17" 
2021-09-14 LTS)
   
   Add `extraJavaTestArgs` to `scalatest-maven-plugin` in `sql/hive/pom.xml`  
first and use Java 17 to run the following two commands:
    
   ```
   build/mvn clean install -Phadoop-3.2 -Phive-2.3 -Phive -pl sql/hive -am 
-DskipTests
   build/mvn test -Phadoop-3.2 -Phive-2.3 -Phive -pl sql/hive
   ```
   
   **Before**
   
   ```
   Run completed in 1 hour, 2 minutes, 3 seconds.
   Total number of tests run: 3547
   Suites: completed 117, aborted 0
   Tests: succeeded 3525, failed 22, canceled 6, ignored 605, pending 0
   *** 22 TESTS FAILED ***
   ```
   
   **After**
   
   ```
   Run completed in 1 hour, 4 minutes, 41 seconds.
   Total number of tests run: 3547
   Suites: completed 117, aborted 0
   Tests: succeeded 3547, failed 0, canceled 6, ignored 605, pending 0
   All tests passed.
   ```


-- 
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to