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

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


The following commit(s) were added to refs/heads/branch-3.5 by this push:
     new 4e4d9f07d095 [SPARK-47022][CONNECT][TESTS][3.5] Fix 
`connect/client/jvm` to have explicit `commons-(io|lang3)` test dependency
4e4d9f07d095 is described below

commit 4e4d9f07d0954357e85a6e2b0da47746a4b08501
Author: Dongjoon Hyun <dh...@apple.com>
AuthorDate: Sun Feb 11 14:38:48 2024 -0800

    [SPARK-47022][CONNECT][TESTS][3.5] Fix `connect/client/jvm` to have 
explicit `commons-(io|lang3)` test dependency
    
    ### What changes were proposed in this pull request?
    
    This PR aims to add `commons-io` and `commons-lang3` test dependency to 
`connector/client/jvm` module.
    
    ### Why are the changes needed?
    
    `connector/client/jvm` module uses `commons-io` and `commons-lang3` during 
testing like the following.
    
    
https://github.com/apache/spark/blob/9700da7bfc1abb607f3cb916b96724d0fb8f2eba/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/ClientE2ETestSuite.scala#L26-L28
    
    Currently, it's broken due to that.
    
    - https://github.com/apache/spark/actions?query=branch%3Abranch-3.5
    
    ### Does this PR introduce _any_ user-facing change?
    
    No, this is a test-dependency only change.
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #45081 from dongjoon-hyun/SPARK-47022.
    
    Authored-by: Dongjoon Hyun <dh...@apple.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 connector/connect/client/jvm/pom.xml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/connector/connect/client/jvm/pom.xml 
b/connector/connect/client/jvm/pom.xml
index 236e5850b762..0c0d4cdad3a9 100644
--- a/connector/connect/client/jvm/pom.xml
+++ b/connector/connect/client/jvm/pom.xml
@@ -71,6 +71,16 @@
       <version>${ammonite.version}</version>
       <scope>provided</scope>
     </dependency>
+    <dependency>
+      <groupId>commons-io</groupId>
+      <artifactId>commons-io</artifactId>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.commons</groupId>
+      <artifactId>commons-lang3</artifactId>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.scalacheck</groupId>
       <artifactId>scalacheck_${scala.binary.version}</artifactId>


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

Reply via email to