yangjf2019 commented on a change in pull request #9532: [hotfix][docs] Fix the 
dependent version of the hive connector error .
URL: https://github.com/apache/flink/pull/9532#discussion_r319818490
 
 

 ##########
 File path: docs/dev/table/hive/index.md
 ##########
 @@ -78,8 +78,8 @@ To integrate with Hive, users need the following 
dependencies in their project.
 
 <dependency>
   <groupId>org.apache.flink</groupId>
-  <artifactId>flink-shaded-hadoop-2-uber-2.7.5-{{site.version}}</artifactId>
-  <version>{{site.version}}</version>
+  <artifactId>flink-shaded-hadoop-2-uber</artifactId>
+  <version>2.7.5-7.0</version>
 
 Review comment:
   Hi, @wuchong @zentol ,I compiled the latest code, however we do not find the 
maven dependency as follows. 
   `<dependency>
     <groupId>org.apache.flink</groupId>
     <artifactId>flink-shaded-hadoop-2-uber</artifactId>
     <version>2.6.5-1.10-SNAPSHOT</version>
     <scope>provided</scope>
   </dependency>
   `
   `<dependency>
     <groupId>org.apache.flink</groupId>
     <artifactId>flink-hadoop-compatibility_1.10-SNAPSHOT</artifactId>
     <version>1.10-SNAPSHOT</version>
     <scope>provided</scope>
   </dependency>`
   so I think this 
[PR](https://github.com/apache/flink/commit/614e9b73345c3ec09d55f75426968b7329c7953a)
 is problematic. Many users will ask why this maven dependency is not found.And 
I saw that 
[‘travis-ci’](https://travis-ci.org/apache/flink/builds/578111960?utm_source=github_status&utm_medium=notification)
 has failed to build. Why can it be merged into master?
   I have submitted a new [Pull 
Request](https://github.com/apache/flink/pull/9591), please take a look, thank 
you!

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to