Skip to content

Commit f3abee3

Browse files
committed
[SPARK-30051][BUILD] Clean up hadoop-3.2 dependency
### What changes were proposed in this pull request? This PR aims to cut `org.eclipse.jetty:jetty-webapp`and `org.eclipse.jetty:jetty-xml` transitive dependency from `hadoop-common`. ### Why are the changes needed? This will simplify our dependency management by the removal of unused dependencies. ### Does this PR introduce any user-facing change? No. ### How was this patch tested? Pass the GitHub Action with all combinations and the Jenkins UT with (Hadoop-3.2). Closes apache#26742 from dongjoon-hyun/SPARK-30051. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent 60f20e5 commit f3abee3

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

dev/deps/spark-deps-hadoop-3.2-hive-2.3

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,8 +135,6 @@ jersey-container-servlet-core-2.29.jar
135135
jersey-hk2-2.29.jar
136136
jersey-media-jaxb-2.29.jar
137137
jersey-server-2.29.jar
138-
jetty-webapp-9.4.18.v20190429.jar
139-
jetty-xml-9.4.18.v20190429.jar
140138
jline-2.14.6.jar
141139
joda-time-2.10.5.jar
142140
jodd-core-3.5.2.jar

pom.xml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -406,6 +406,12 @@
406406
<version>${jetty.version}</version>
407407
<scope>provided</scope>
408408
</dependency>
409+
<dependency>
410+
<groupId>org.eclipse.jetty</groupId>
411+
<artifactId>jetty-webapp</artifactId>
412+
<version>${jetty.version}</version>
413+
<scope>provided</scope>
414+
</dependency>
409415
<dependency>
410416
<groupId>com.google.guava</groupId>
411417
<artifactId>guava</artifactId>
@@ -1030,6 +1036,10 @@
10301036
<groupId>javax.ws.rs</groupId>
10311037
<artifactId>jsr311-api</artifactId>
10321038
</exclusion>
1039+
<exclusion>
1040+
<groupId>org.eclipse.jetty</groupId>
1041+
<artifactId>jetty-webapp</artifactId>
1042+
</exclusion>
10331043
</exclusions>
10341044
</dependency>
10351045
<dependency>

0 commit comments

Comments
 (0)