Skip to content

Commit

Permalink
[FLINK-10984][build] Remove flink-shaded-hadoop(-uber) modules
Browse files Browse the repository at this point in the history
  • Loading branch information
sunjincheng121 authored and zentol committed Jun 19, 2019
1 parent b1d5bf9 commit ee4a2f7
Show file tree
Hide file tree
Showing 35 changed files with 102 additions and 1,795 deletions.
4 changes: 2 additions & 2 deletions flink-connectors/flink-connector-filesystem/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ under the License.

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-connectors/flink-hadoop-compatibility/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,8 @@ under the License.

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-connectors/flink-hbase/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -110,8 +110,8 @@ under the License.

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-connectors/flink-hcatalog/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ under the License.

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-connectors/flink-orc/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ under the License.
<!-- Replacement for ORC's Hadoop dependency. -->
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
11 changes: 3 additions & 8 deletions flink-dist/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ under the License.
<exclusions>
<exclusion>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<artifactId>flink-shaded-hadoop-2</artifactId>
</exclusion>
</exclusions>
</dependency>
Expand Down Expand Up @@ -425,15 +425,10 @@ under the License.
The Hadoop 2 Uber jar should not go into the Flink dist jar, but
sit next to it. Hence, we set it to 'provided' here.
-->

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2-uber</artifactId>
<version>${hadoop.version}-${project.version}</version>
<!--
Exclusion of flink-shaded-hadoop2 not necessary, dependencies
are shaded away properly by flink-shaded-hadoop2-uber.
-->
<artifactId>flink-shaded-hadoop-2-uber</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>
</dependencies>
Expand Down
17 changes: 9 additions & 8 deletions flink-dist/src/main/assemblies/hadoop.xml
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,14 @@ under the License.
<includeBaseDirectory>true</includeBaseDirectory>
<baseDirectory>flink-${project.version}</baseDirectory>

<files>
<!-- copy the Hadoop uber jar -->
<file>
<source>../flink-shaded-hadoop/flink-shaded-hadoop2-uber/target/flink-shaded-hadoop2-uber-${hadoop.version}-${project.version}.jar</source>
<dependencySets>
<dependencySet>
<outputDirectory>lib/</outputDirectory>
<destName>flink-shaded-hadoop2-uber-${project.version}-${hadoop.version}.jar</destName>
<fileMode>0644</fileMode>
</file>
</files>
<useTransitiveDependencies>true</useTransitiveDependencies>
<scope>provided</scope>
<includes>
<include>org.apache.flink:flink-shaded-hadoop-2-uber</include>
</includes>
</dependencySet>
</dependencySets>
</assembly>
4 changes: 2 additions & 2 deletions flink-end-to-end-tests/flink-bucketing-sink-test/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@
</dependency>
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
<exclusions>
<!-- Needed for proper dependency convergence -->
Expand Down
4 changes: 2 additions & 2 deletions flink-filesystems/flink-hadoop-fs/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ under the License.

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<optional>true</optional>
</dependency>

Expand Down
2 changes: 1 addition & 1 deletion flink-filesystems/flink-swift-fs-hadoop/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ under the License.
<exclusions>
<exclusion>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<artifactId>flink-shaded-hadoop-2</artifactId>
</exclusion>
</exclusions>
</dependency>
Expand Down
4 changes: 2 additions & 2 deletions flink-formats/flink-parquet/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,8 @@ under the License.
<!-- Hadoop is needed by Parquet -->
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-formats/flink-sequence-file/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,8 @@ under the License.
<!-- Hadoop is needed for SequenceFile -->
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>provided</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-fs-tests/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,8 @@ under the License.
<dependencies>
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<scope>test</scope>
</dependency>

Expand Down
4 changes: 2 additions & 2 deletions flink-runtime/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,8 @@ under the License.

<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-hadoop2</artifactId>
<version>${hadoop.version}-${project.version}</version>
<artifactId>flink-shaded-hadoop-2</artifactId>
<version>${hadoop.version}-${flink.shaded.version}</version>
<optional>true</optional>
</dependency>

Expand Down
117 changes: 0 additions & 117 deletions flink-shaded-hadoop/flink-shaded-hadoop2-uber/pom.xml

This file was deleted.

This file was deleted.

Loading

0 comments on commit ee4a2f7

Please sign in to comment.