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

yangjie01 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 2723bb11e3c7 [SPARK-46476][BUILD][CORE][CONNECT] Move `IvyTestUtils` 
back to `src/test` directory
2723bb11e3c7 is described below

commit 2723bb11e3c7b9e2252e26cca904e2e53aaa68fb
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Thu Dec 21 20:50:34 2023 +0800

    [SPARK-46476][BUILD][CORE][CONNECT] Move `IvyTestUtils` back to `src/test` 
directory
    
    ### What changes were proposed in this pull request?
    This pr move `IvyTestUtils` back to `src/test` directory because it has 
been in the `src/test` directory before the refactoring work of 
https://github.com/apache/spark/pull/43354.
    
    Meanwhile, in order to make the `core` and `connect-client-jvm` module use 
`IvyTestUtils` in the tests, this pr has added the corresponding Maven 
dependencies in the respective `pom.xml` files.
    
    ### Why are the changes needed?
    Move `IvyTestUtils` back to `src/test` directory
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    Pass GitHub Actions
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #44440 from LuciferYang/mv-IvyTestUtils-to-test-dir.
    
    Authored-by: yangjie01 <yangji...@baidu.com>
    Signed-off-by: yangjie01 <yangji...@baidu.com>
---
 .../{main => test}/scala/org/apache/spark/util/IvyTestUtils.scala  | 0
 connector/connect/client/jvm/pom.xml                               | 7 +++++++
 core/pom.xml                                                       | 7 +++++++
 3 files changed, 14 insertions(+)

diff --git 
a/common/utils/src/main/scala/org/apache/spark/util/IvyTestUtils.scala 
b/common/utils/src/test/scala/org/apache/spark/util/IvyTestUtils.scala
similarity index 100%
rename from common/utils/src/main/scala/org/apache/spark/util/IvyTestUtils.scala
rename to common/utils/src/test/scala/org/apache/spark/util/IvyTestUtils.scala
diff --git a/connector/connect/client/jvm/pom.xml 
b/connector/connect/client/jvm/pom.xml
index eb98e4203a96..8057a33df178 100644
--- a/connector/connect/client/jvm/pom.xml
+++ b/connector/connect/client/jvm/pom.xml
@@ -82,6 +82,13 @@
       <artifactId>scalacheck_${scala.binary.version}</artifactId>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.apache.spark</groupId>
+      <artifactId>spark-common-utils_${scala.binary.version}</artifactId>
+      <version>${project.version}</version>
+      <classifier>tests</classifier>
+      <scope>test</scope>
+    </dependency>
     <!-- Use mima to perform the compatibility check -->
     <dependency>
       <groupId>com.typesafe</groupId>
diff --git a/core/pom.xml b/core/pom.xml
index f2d1f4eb144a..c093213bd6b9 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -451,6 +451,13 @@
       <classifier>tests</classifier>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.apache.spark</groupId>
+      <artifactId>spark-common-utils_${scala.binary.version}</artifactId>
+      <version>${project.version}</version>
+      <classifier>tests</classifier>
+      <scope>test</scope>
+    </dependency>
 
     <!--
       This spark-tags test-dep is needed even though it isn't used in this 
module, otherwise testing-cmds that exclude


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

Reply via email to