Github user StephanEwen commented on the pull request:
https://github.com/apache/incubator-flink/pull/102#issuecomment-53700461
Looks good so far, I would like to merge this soon.
One question, though: Have you looked into the new dependencies (redis,
memcached, ...) and checked whether they have notice files with notices?
If so, the notices may need to go into
`flink-dist/src/main/flink-bin/NOTICE`.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
