Skip to content

Commit

Permalink
[SPARK-49964][BUILD] Remove ws-rs-api package
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

- To Remove the dependency of `javax.ws.rs.ws-rs-api` as it's no longer required.

Prior discussion can be found on:
 - apache#41340
 - apache#45154

### Why are the changes needed?
In the past, the codebase used to have a few .scala classes referencing and using the `ws-rs-api`, such as apache@b7fdc23#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d8R624-R627

However as the time passed by, all usages of `ws-rs-api` are either got removed / refactored. Hence there is no need to have it import on root POM as now and we can always re-introduce it later, if the usage can be justified again.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Unit-test, to make sure the codebase is not impacted by the removal of the dependency.

### Was this patch authored or co-authored using generative AI tooling?
No

Closes apache#48461 from hiufung-kwok/ft-hf-SPARK-49963-remove-ws-rs-api.

Authored-by: HiuFung Kwok <hiufung.kwok.852@gmail.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
  • Loading branch information
hiufung-kwok authored and himadripal committed Oct 19, 2024
1 parent ad81dc5 commit 31038c3
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -1115,11 +1115,6 @@
<artifactId>jersey-client</artifactId>
<version>${jersey.version}</version>
</dependency>
<dependency>
<groupId>javax.ws.rs</groupId>
<artifactId>javax.ws.rs-api</artifactId>
<version>2.0.1</version>
</dependency>
<dependency>
<groupId>javax.xml.bind</groupId>
<artifactId>jaxb-api</artifactId>
Expand Down

0 comments on commit 31038c3

Please sign in to comment.