luoyuxia commented on code in PR #21128:
URL: https://github.com/apache/flink/pull/21128#discussion_r1002865693
##########
flink-connectors/flink-connector-hive/pom.xml:
##########
@@ -39,6 +39,11 @@ under the License.
<reflections.version>0.9.8</reflections.version>
<derby.version>10.10.2.0</derby.version>
<hive.avro.version>1.8.2</hive.avro.version>
+ <!--
+ Hive 2.3.4 relies on Hadoop 2.7.2 and later versions.
+ For Hadoop 2.7, the minor Hadoop version supported for
flink-shaded-hadoop-2-uber is 2.7.5
+ -->
+
<hivemetastore.hadoop.version>2.7.5</hivemetastore.hadoop.version>
Review Comment:
Although the Hive release page has said it works with Hadoop 2.x.y, it
seems it will still has some problems in lower hadoop version as tracked in
FLINK-13139.
But I think if we use the the Hadoop version from the parent POM, we can
remove the references to `hivemetastore.hadoop.version` since we will use a
higher Hadoop version.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
