[
https://issues.apache.org/jira/browse/FLINK-5690?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15864266#comment-15864266
]
Robert Metzger commented on FLINK-5690:
---------------------------------------
1) I'll open a pull request.
Regarding 2) This is something we could look into.
However, so far, almost all complaints from users were about the dependencies
we inherit from Hadoop. That's why I've tried relocating all of Hadoop's
dependencies:
https://issues.apache.org/jira/browse/FLINK-5297?focusedCommentId=15815050&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15815050
(with mixed results).
Also, I'm planning to provide a "hadoop free" version of Flink. As more and
more of our users actually run without anything from Hadoop (for example on AWS
with S3 and Docker), these users don't need all the deps we inherit from Hadoop.
The libraries you've mentioned are pretty good with maintaining API
compatibility. So you can mix these libraries better in one classpath.
> protobuf is not shaded properly
> -------------------------------
>
> Key: FLINK-5690
> URL: https://issues.apache.org/jira/browse/FLINK-5690
> Project: Flink
> Issue Type: Bug
> Components: Build System
> Affects Versions: 1.1.4, 1.3.0
> Reporter: Andrey
> Assignee: Robert Metzger
>
> Currently distributive contains com/google/protobuf package. Without proper
> shading client code could fail with:
> {code}
> Caused by: java.lang.IllegalAccessError: tried to access method
> com.google.protobuf.XXXX
> {code}
> Steps to reproduce:
> * create job class "com.google.protobuf.TestClass"
> * call com.google.protobuf.TextFormat.escapeText(String) method from this
> class
> * deploy job to flink cluster (usign web console for example)
> * run job. In logs IllegalAccessError.
> Issue in package protected method and different classloaders. TestClass
> loaded by FlinkUserCodeClassLoader, but TextFormat class loaded by
> sun.misc.Launcher$AppClassLoader
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
