This is an automated email from the ASF dual-hosted git repository. twalthr pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/master by this push: new a5547fa [FLINK-22813][connector-hive] Clean up flink-connector-hive's pom.xml a5547fa is described below commit a5547fad3897c6fd96238787a0e66774d7aad3d2 Author: Timo Walther <twal...@apache.org> AuthorDate: Tue Jun 1 10:05:15 2021 +0200 [FLINK-22813][connector-hive] Clean up flink-connector-hive's pom.xml This closes #16038. --- flink-connectors/flink-connector-hive/pom.xml | 132 +++++++++++++------------- 1 file changed, 65 insertions(+), 67 deletions(-) diff --git a/flink-connectors/flink-connector-hive/pom.xml b/flink-connectors/flink-connector-hive/pom.xml index 985676c..f6937f3 100644 --- a/flink-connectors/flink-connector-hive/pom.xml +++ b/flink-connectors/flink-connector-hive/pom.xml @@ -129,7 +129,7 @@ under the License. <dependencies> - <!-- core dependencies --> + <!-- Table ecosystem --> <dependency> <groupId>org.apache.flink</groupId> @@ -140,29 +140,31 @@ under the License. <dependency> <groupId>org.apache.flink</groupId> - <artifactId>flink-hadoop-compatibility_${scala.binary.version}</artifactId> + <artifactId>flink-table-api-java-bridge_${scala.binary.version}</artifactId> <version>${project.version}</version> + <scope>provided</scope> </dependency> <dependency> <groupId>org.apache.flink</groupId> - <artifactId>flink-table-api-java-bridge_${scala.binary.version}</artifactId> + <artifactId>flink-table-runtime-blink_${scala.binary.version}</artifactId> <version>${project.version}</version> <scope>provided</scope> </dependency> <dependency> <groupId>org.apache.flink</groupId> - <artifactId>flink-table-runtime-blink_${scala.binary.version}</artifactId> + <artifactId>flink-table-planner-blink_${scala.binary.version}</artifactId> <version>${project.version}</version> <scope>provided</scope> </dependency> + <!-- Connectors --> + <dependency> <groupId>org.apache.flink</groupId> - <artifactId>flink-table-planner-blink_${scala.binary.version}</artifactId> + <artifactId>flink-hadoop-compatibility_${scala.binary.version}</artifactId> <version>${project.version}</version> - <scope>provided</scope> </dependency> <dependency> @@ -177,7 +179,7 @@ under the License. <version>${project.version}</version> </dependency> - <!-- format dependencies --> + <!-- Formats --> <dependency> <groupId>org.apache.flink</groupId> @@ -209,16 +211,7 @@ under the License. <version>${project.version}</version> </dependency> - <!--We downgrade the version of avro for tests. We do that because we test Hive's built-in - formats, which do not work with newer avro versions. We do not downgrade the version - permanently, because it should be safe to use hive as a catalog with newer avro - versions. --> - <dependency> - <groupId>org.apache.avro</groupId> - <artifactId>avro</artifactId> - <version>${hive.avro.version}</version> - <scope>test</scope> - </dependency> + <!-- File systems --> <dependency> <groupId>org.apache.flink</groupId> @@ -226,7 +219,8 @@ under the License. <version>${project.version}</version> </dependency> - <!-- Hadoop dependency --> + <!-- Hadoop --> + <!-- Hadoop as provided dependencies, so we can depend on them without pulling in Hadoop --> <!-- @@ -245,7 +239,9 @@ under the License. <artifactId>hadoop-mapreduce-client-core</artifactId> <scope>provided</scope> </dependency> - <!-- Hive dependencies --> + + <!-- Hive --> + <!-- Note: Hive published jars do not have proper dependencies declared. We need to push for HIVE-16391 (https://issues.apache.org/jira/browse/HIVE-16391) to resolve this problem. --> @@ -515,7 +511,7 @@ under the License. </exclusions> </dependency> - <!-- test dependencies --> + <!-- Tests --> <dependency> <groupId>org.apache.flink</groupId> @@ -535,13 +531,6 @@ under the License. <dependency> <groupId>org.apache.flink</groupId> - <artifactId>flink-table-planner_${scala.binary.version}</artifactId> - <version>${project.version}</version> - <scope>test</scope> - </dependency> - - <dependency> - <groupId>org.apache.flink</groupId> <artifactId>flink-table-planner-blink_${scala.binary.version}</artifactId> <version>${project.version}</version> <type>test-jar</type> @@ -549,7 +538,6 @@ under the License. </dependency> <!--flink-java and flink-clients test dependencies used for HiveInputFormatTest--> - <dependency> <groupId>org.apache.flink</groupId> <artifactId>flink-java</artifactId> @@ -564,27 +552,27 @@ under the License. </dependency> <dependency> - <groupId>com.klarna</groupId> - <artifactId>hiverunner</artifactId> - <version>${hiverunner.version}</version> - <scope>test</scope> - <exclusions> - <exclusion> - <groupId>org.apache.hive</groupId> - <artifactId>hive-serde</artifactId> - </exclusion> - <exclusion> - <groupId>org.apache.hive</groupId> - <artifactId>hive-jdbc</artifactId> - </exclusion> - <exclusion> - <groupId>org.apache.hive</groupId> - <artifactId>hive-service</artifactId> - </exclusion> - <exclusion> - <groupId>org.apache.hive</groupId> - <artifactId>hive-contrib</artifactId> - </exclusion> + <groupId>com.klarna</groupId> + <artifactId>hiverunner</artifactId> + <version>${hiverunner.version}</version> + <scope>test</scope> + <exclusions> + <exclusion> + <groupId>org.apache.hive</groupId> + <artifactId>hive-serde</artifactId> + </exclusion> + <exclusion> + <groupId>org.apache.hive</groupId> + <artifactId>hive-jdbc</artifactId> + </exclusion> + <exclusion> + <groupId>org.apache.hive</groupId> + <artifactId>hive-service</artifactId> + </exclusion> + <exclusion> + <groupId>org.apache.hive</groupId> + <artifactId>hive-contrib</artifactId> + </exclusion> <exclusion> <groupId>org.apache.hive</groupId> <artifactId>hive-exec</artifactId> @@ -658,8 +646,8 @@ under the License. <groupId>org.slf4j</groupId> <artifactId>slf4j-log4j12</artifactId> </exclusion> - </exclusions> - </dependency> + </exclusions> + </dependency> <!-- We have 0.9.10 in dependency management but hiverunner requires 0.9.8, so need to explicitly specify it here --> <dependency> @@ -676,10 +664,10 @@ under the License. </dependency> <dependency> - <groupId>org.apache.hive</groupId> - <artifactId>hive-service</artifactId> - <version>${hive.version}</version> - <scope>test</scope> + <groupId>org.apache.hive</groupId> + <artifactId>hive-service</artifactId> + <version>${hive.version}</version> + <scope>test</scope> <exclusions> <exclusion> <groupId>org.apache.hive</groupId> @@ -762,10 +750,10 @@ under the License. </dependency> <dependency> - <groupId>org.apache.hive.hcatalog</groupId> - <artifactId>hive-hcatalog-core</artifactId> - <version>${hive.version}</version> - <scope>test</scope> + <groupId>org.apache.hive.hcatalog</groupId> + <artifactId>hive-hcatalog-core</artifactId> + <version>${hive.version}</version> + <scope>test</scope> <exclusions> <exclusion> <groupId>org.apache.hive</groupId> @@ -811,8 +799,8 @@ under the License. <groupId>org.slf4j</groupId> <artifactId>slf4j-log4j12</artifactId> </exclusion> - </exclusions> - </dependency> + </exclusions> + </dependency> <dependency> <groupId>org.apache.hive.hcatalog</groupId> @@ -852,11 +840,11 @@ under the License. </dependency> <dependency> - <groupId>org.apache.flink</groupId> - <artifactId>flink-test-utils_${scala.binary.version}</artifactId> - <version>${project.version}</version> + <groupId>org.apache.flink</groupId> + <artifactId>flink-test-utils_${scala.binary.version}</artifactId> + <version>${project.version}</version> <scope>test</scope> - </dependency> + </dependency> <!-- The derby we use suffers some security vulnerabilities. Explicitly set it to test scope here. --> <dependency> @@ -866,7 +854,17 @@ under the License. <scope>test</scope> </dependency> - </dependencies> + <!--We downgrade the version of avro for tests. We do that because we test Hive's built-in + formats, which do not work with newer avro versions. We do not downgrade the version + permanently, because it should be safe to use hive as a catalog with newer avro + versions. --> + <dependency> + <groupId>org.apache.avro</groupId> + <artifactId>avro</artifactId> + <version>${hive.avro.version}</version> + <scope>test</scope> + </dependency> + </dependencies> <build> <plugins> @@ -1109,7 +1107,7 @@ under the License. <properties> <hive.version>3.1.1</hive.version> <derby.version>10.14.1.0</derby.version> - <!-- need a hadoop version that fixes HADOOP-14683 --> + <!-- need a hadoop version that fixes HADOOP-14683 --> <hivemetastore.hadoop.version>2.8.2</hivemetastore.hadoop.version> </properties>