You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2025/01/30 17:08:00 UTC

(spark) branch branch-4.0 updated: [SPARK-51041][BUILD] Add `hive-llap-client` and `hive-llap-common` as test dependency of `hive-thriftserver`

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

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


The following commit(s) were added to refs/heads/branch-4.0 by this push:
     new 99df4a937e5b [SPARK-51041][BUILD] Add `hive-llap-client` and `hive-llap-common` as test dependency of `hive-thriftserver`
99df4a937e5b is described below

commit 99df4a937e5b401c49c7c33f4986991cd6557e8d
Author: yangjie01 <ya...@baidu.com>
AuthorDate: Thu Jan 30 09:07:41 2025 -0800

    [SPARK-51041][BUILD] Add `hive-llap-client` and `hive-llap-common` as test dependency of `hive-thriftserver`
    
    ### What changes were proposed in this pull request?
    This pr aims to add `hive-llap-client` and `hive-llap-common` as test dependency of `hive-thriftserver`
    
    ### Why are the changes needed?
    Fix  maven test of  `hive-thriftserver`.  Due to the lack of these test dependencies, testing `hive-thriftserver` using Maven will  hang.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    - Pass GitHub Actions
    - Manually check: After adding the test dependencies, when testing the `hive-thriftserver` module using Maven, `HiveThriftBinaryServerSuite` will no longer hang.
    
    ```
    build/mvn -Pyarn -Pkubernetes -Pvolcano -Phive -Phive-thriftserver -Phadoop-cloud -Pjvm-profiler -Pspark-ganglia-lgpl -Pkinesis-asl clean install
    build/mvn -pl sql/hive-thriftserver -Pyarn -Pkubernetes -Pvolcano -Phive -Phive-thriftserver -Phadoop-cloud -Pjvm-profiler -Pspark-ganglia-lgpl -Pkinesis-asl clean install -fae
    ```
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #49736 from LuciferYang/SPARK-51041.
    
    Authored-by: yangjie01 <ya...@baidu.com>
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
    (cherry picked from commit ecf6851a97c01976f49ca04be797b113cddc684a)
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
 sql/hive-thriftserver/pom.xml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/sql/hive-thriftserver/pom.xml b/sql/hive-thriftserver/pom.xml
index d066e235ebea..ad5567bb1bdf 100644
--- a/sql/hive-thriftserver/pom.xml
+++ b/sql/hive-thriftserver/pom.xml
@@ -148,6 +148,16 @@
       <artifactId>byte-buddy-agent</artifactId>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>${hive.group}</groupId>
+      <artifactId>hive-llap-common</artifactId>
+      <scope>${hive.llap.scope}</scope>
+    </dependency>
+    <dependency>
+      <groupId>${hive.group}</groupId>
+      <artifactId>hive-llap-client</artifactId>
+      <scope>${hive.llap.scope}</scope>
+    </dependency>
     <dependency>
       <groupId>net.sf.jpam</groupId>
       <artifactId>jpam</artifactId>


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