Github user hsaputra commented on the pull request:
https://github.com/apache/flink/pull/683#issuecomment-103135620
Thanks all, merging ...
---
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 thi
Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/683
---
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 enabl
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/683#issuecomment-103062521
Good to merge...
---
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 fe
Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/683#issuecomment-102944771
LGTM
---
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 an
Github user hsaputra commented on the pull request:
https://github.com/apache/flink/pull/683#issuecomment-102935021
If no comment, will merge this simple fix end of day tomorrow.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user hsaputra opened a pull request:
https://github.com/apache/flink/pull/683
Wrap very long line in FlatMapOperator class
As previous discussions before, we do not restrict max chars per line for
Java code, but this one just way too long compare to others that I just think