Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/1174#issuecomment-142753142
@hsaputra I think the wiki page is not correct any more. Netty was
discussed to be shaded, but was never added to the relocated artifacts.
That was a good decision, IMHO. Dependencies that upgrade well-behaved need
not (and in most cases should not) be shaded. Netty behaved well so far: When
the changed API and behavior, they used a different namespace. Same is true for
the Apache Commons libraries, for example the switch from commons lang version
2 (in the old namespace) to commons lang version 3 (new package name and
artifact id).
Unfortunately, same is not true for Guava, Joda Time, ASM, Protobuf, ...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
