[
https://issues.apache.org/jira/browse/FLINK-7973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16243705#comment-16243705
]
ASF GitHub Bot commented on FLINK-7973:
---------------------------------------
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/4961#discussion_r149635214
--- Diff: flink-filesystems/flink-s3-fs-presto/README.md ---
@@ -0,0 +1,28 @@
+This project is a wrapper around the S3 file system from the Presto
project which shades all dependencies.
+Initial simple tests seem to indicate that it responds slightly faster
+and in a bit more lightweight manner to write/read/list requests, compared
+to the Hadoop s3a FS, but it has some semantic differences.
+
+We also relocate the shaded Hadoop version to allow running in a different
+setup. For this to work, however, we needed to adapt Hadoop's
`Configuration`
+class to load a (shaded) `core-default-shaded.xml` configuration with the
+relocated class names of classes loaded via reflection
+(in the fute, we may need to extend this to `mapred-default.xml` and
`hdfs-defaults.xml` and their respective configuration classes).
+
+# Changing the Hadoop Version
+
+If you want to change the Hadoop version this project depends on, the
following
+steps are required to keep the shading correct:
+
+1. copy `org/apache/hadoop/conf/Configuration.java` from the respective
Hadoop jar file (from `com.facebook.presto.hadoop/hadoop-apache2`) to this
project
+ - adapt the `Configuration` class by replacing `core-default.xml` with
`core-default-shaded.xml`.
+2. copy `core-default.xml` from the respective Hadoop jar (from
`com.facebook.presto.hadoop/hadoop-apache2`) file to this project as
+ - `src/main/resources/core-default-shaded.xml` (replacing every
occurence of `org.apache.hadoop` with
`org.apache.flink.fs.s3presto.shaded.org.apache.hadoop`)
+ - `src/test/resources/core-site.xml` (as is)
+3. verify the shaded jar:
+ - does not contain any unshaded classes except for
`org.apache.flink.fs.s3presto.S3FileSystemFactory`
+ - every other classes should be under
`org.apache.flink.fs.s3presto.shaded`
--- End diff --
"every other class" or "all other classes"
> Fix service shading relocation for S3 file systems
> --------------------------------------------------
>
> Key: FLINK-7973
> URL: https://issues.apache.org/jira/browse/FLINK-7973
> Project: Flink
> Issue Type: Bug
> Reporter: Stephan Ewen
> Assignee: Nico Kruber
> Priority: Blocker
> Fix For: 1.4.0
>
>
> The shade plugin relocates services incorrectly currently, applying
> relocation patterns multiple times.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
