This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 1292b566550 [SPARK-44852][BUILD] Exclude `junit-jupiter-api` from 
`curator-test`
1292b566550 is described below

commit 1292b566550235e29b7a81f7c0dc022b33c92c2f
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Fri Aug 18 09:04:47 2023 -0700

    [SPARK-44852][BUILD] Exclude `junit-jupiter-api` from `curator-test`
    
    ### What changes were proposed in this pull request?
    This PR excludes `junit-jupiter-api` from `curator-test` to avoid maven 
test wrongly using JUnit 5 for testing.
    
    ### Why are the changes needed?
    [SPARK-44792](https://issues.apache.org/jira/browse/SPARK-44792) Upgrade 
curator to 5.2.0 and `curator-test` depends on `junit-jupiter-api`, but Apache 
Spark currently does not support testing with JUnit5, so it will report the 
following error when executing
    
    ```
    build/mvn clean install -pl core -am 
-Dtest.exclude.tags=org.apache.spark.tags.ExtendedLevelDBTest
    ```
    
    ```
    [INFO] -------------------------------------------------------
    [INFO]  T E S T S
    [INFO] -------------------------------------------------------
    [ERROR] TestEngine with ID 'junit-vintage' failed to discover tests
    [INFO]
    [INFO] Results:
    [INFO]
    [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0
    [INFO]
    [INFO] 
------------------------------------------------------------------------
    [INFO] Reactor Summary for Spark Project Parent POM 4.0.0-SNAPSHOT:
    [INFO]
    [INFO] Spark Project Parent POM ........................... SUCCESS [  
4.631 s]
    [INFO] Spark Project Tags ................................. SUCCESS [  
9.044 s]
    [INFO] Spark Project Local DB ............................. SUCCESS [ 
12.686 s]
    [INFO] Spark Project Common Utils ......................... SUCCESS [ 
12.216 s]
    [INFO] Spark Project Networking ........................... SUCCESS [ 
54.368 s]
    [INFO] Spark Project Shuffle Streaming Service ............ SUCCESS [ 
14.355 s]
    [INFO] Spark Project Unsafe ............................... SUCCESS [ 
12.321 s]
    [INFO] Spark Project Launcher ............................. SUCCESS [ 
10.019 s]
    [INFO] Spark Project Core ................................. FAILURE [01:06 
min]
    [INFO] 
------------------------------------------------------------------------
    [INFO] BUILD FAILURE
    [INFO] 
------------------------------------------------------------------------
    [INFO] Total time:  03:16 min
    [INFO] Finished at: 2023-08-17T23:30:36+08:00
    [INFO] 
------------------------------------------------------------------------
    [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:3.1.2:test (default-test) on 
project spark-core_2.12:
    [ERROR]
    [ERROR] Please refer to 
/Users/yangjie01/SourceCode/git/spark-mine-12/core/target/surefire-reports for 
the individual test results.
    [ERROR] Please refer to dump files (if any exist) [date].dump, 
[date]-jvmRun[N].dump and [date].dumpstream.
    [ERROR] There was an error in the forked process
    [ERROR] TestEngine with ID 'junit-vintage' failed to discover tests
    [ERROR] org.apache.maven.surefire.booter.SurefireBooterForkException: There 
was an error in the forked process
    [ERROR] TestEngine with ID 'junit-vintage' failed to discover tests
    [ERROR]     at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter.fork(ForkStarter.java:628)
    [ERROR]     at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:285)
    [ERROR]     at 
org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:250)
    [ERROR]     at 
org.apache.maven.plugin.surefire.AbstractSurefireMojo.executeProvider(AbstractSurefireMojo.java:1203)
    [ERROR]     at 
org.apache.maven.plugin.surefire.AbstractSurefireMojo.executeAfterPreconditionsChecked(AbstractSurefireMojo.java:1055)
    [ERROR]     at 
org.apache.maven.plugin.surefire.AbstractSurefireMojo.execute(AbstractSurefireMojo.java:871)
    [ERROR]     at 
org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:137)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2(MojoExecutor.java:370)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.MojoExecutor.doExecute(MojoExecutor.java:351)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:215)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:171)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:163)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:117)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:81)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:56)
    [ERROR]     at 
org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:128)
    [ERROR]     at 
org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:299)
    [ERROR]     at 
org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:193)
    [ERROR]     at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:106)
    [ERROR]     at org.apache.maven.cli.MavenCli.execute(MavenCli.java:963)
    [ERROR]     at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:296)
    [ERROR]     at org.apache.maven.cli.MavenCli.main(MavenCli.java:199)
    [ERROR]     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    [ERROR]     at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    [ERROR]     at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    [ERROR]     at java.lang.reflect.Method.invoke(Method.java:498)
    [ERROR]     at 
org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:282)
    [ERROR]     at 
org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:225)
    [ERROR]     at 
org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:406)
    [ERROR]     at 
org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:347)
    [ERROR]
    [ERROR] -> [Help 1]
    [ERROR]
    [ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
    [ERROR] Re-run Maven using the -X switch to enable full debug logging.
    [ERROR]
    [ERROR] For more information about the errors and possible solutions, 
please read the following articles:
    [ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
    [ERROR]
    [ERROR] After correcting the problems, you can resume the build with the 
command
    [ERROR]   mvn <args> -rf :spark-core_2.12
    ```
    
    So we need excludes `junit-jupiter-api` from `curator-test` to restore 
Maven testing.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    - Pass GitHub Actions
    - Manually checked  Maven test no longer has the error message described 
above with this pr
    
    Closes #42539 from LuciferYang/SPARK-44852.
    
    Authored-by: yangjie01 <yangji...@baidu.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 pom.xml | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/pom.xml b/pom.xml
index d27597f5c49..7db8618355f 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1297,6 +1297,13 @@
         <groupId>org.apache.curator</groupId>
         <artifactId>curator-test</artifactId>
         <version>${curator.version}</version>
+        <!-- TODO(SPARK-44170): Remove this `exclusion` after Spark starts 
supporting JUnit5. -->
+        <exclusions>
+          <exclusion>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-api</artifactId>
+          </exclusion>
+        </exclusions>
         <scope>test</scope>
       </dependency>
       <!-- Hadoop 3.x dependencies -->


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to