zentol commented on a change in pull request #10482: [Flink-11956] Unshade s3
filesystems
URL: https://github.com/apache/flink/pull/10482#discussion_r355149149
##########
File path: flink-filesystems/flink-s3-fs-base/pom.xml
##########
@@ -46,14 +46,14 @@ under the License.
<scope>provided</scope>
</dependency>
- <!-- Hadoop's file system abstraction (bundled) -->
- <dependency>
- <groupId>org.apache.flink</groupId>
- <artifactId>flink-fs-hadoop-shaded</artifactId>
- <version>${project.version}</version>
- </dependency>
-
- <!-- The Hadoop file system adapter classes (bundled) -->
+ <!-- Hadoop's file system abstraction (bundled) -->
+ <dependency>
+ <groupId>org.apache.hadoop</groupId>
+ <artifactId>hadoop-common</artifactId>
Review comment:
what's this change about? If the s3 filesystems no longer depend on
`flink-fs-hadoop-shaded`, why are we touching `flink-fs-hadoop-shaded` at all?
Is `flink-fs-hadoop-shaded` even useful on it's own? If not (i.e., it always
being bundled by some other functional component) then the module itself could
probably just be removed.
----------------------------------------------------------------
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
