dmvk commented on a change in pull request #18059:
URL: https://github.com/apache/flink/pull/18059#discussion_r780300834



##########
File path: docs/content/docs/deployment/filesystems/gcs.md
##########
@@ -68,7 +68,7 @@ You must include the following jars in Flink's `lib` 
directory to connect Flink
 </dependency>
 ```
 
-We have tested with `flink-shared-hadoop2-uber` version >= `2.8.3-1.8.3`.
+We have tested with `flink-shared-hadoop2-uber` version >= `2.8.5-1.8.3`.

Review comment:
       This whole section looks really weird :/
   
   First off:
   - The artifact name is incorrect -> `flink-shaded-hadoop2-uber`
   - The shaded hadoop jars are no longer published / supported -> 
https://issues.apache.org/jira/browse/FLINK-17685
   
   I'd suggest to leave this file unchanged as is (so at least it still refers 
to an existing artifact) and I'll create a blocker for 1.15 to update the docs 
for GCS.

##########
File path: docs/content/docs/deployment/filesystems/gcs.md
##########
@@ -68,7 +68,7 @@ You must include the following jars in Flink's `lib` 
directory to connect Flink
 </dependency>
 ```
 
-We have tested with `flink-shared-hadoop2-uber` version >= `2.8.3-1.8.3`.
+We have tested with `flink-shared-hadoop2-uber` version >= `2.8.5-1.8.3`.

Review comment:
       -> https://issues.apache.org/jira/browse/FLINK-25577




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to