Skip to content

Commit

Permalink
[SPARK-51041][BUILD] Add hive-llap-client and hive-llap-common as…
Browse files Browse the repository at this point in the history
… 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 <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
LuciferYang authored and dongjoon-hyun committed Jan 30, 2025
1 parent 994b4b1 commit ecf6851
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions sql/hive-thriftserver/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down

0 comments on commit ecf6851

Please sign in to comment.