Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/1871
---
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 enab
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/1871#issuecomment-209407499
Looks good. Failed test seems to be a Scala compiler issue, not related to
this pull request.
Merging this...
---
If your project is set up for it, you can
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/1871#issuecomment-209062546
Code looks good, but it triggers a checkstyle violation: indentation
includes a space.
Please have a look at the Travis CI logs...
---
If your project is set up
GitHub user skyahead opened a pull request:
https://github.com/apache/flink/pull/1871
[FLINK-3732] [Core] Potential null deference in ExecutionConfig#equals()
Thanks for contributing to Apache Flink. Before you open your pull request,
please take the following check list into consid
