Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/3000
First of all, congratulations on scoring pull request number 3000!
This looks good to me!
Few things to double check
- I was assuming that `flink-avro` is actually part of the `flink-dist`
fat jar already. Should it still be in `opt`?
- `flink-hcatalog` does not have a Scala version suffix, probably should
have one.
- Do you know if the proper relocation of classes from Guava, Asm,
Curator, etc, happening in these jars?
---
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.
---
