[GitHub] flink pull request: Wrap very long line in FlatMapOperator class

2015-05-18 Thread hsaputra
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] flink pull request: Wrap very long line in FlatMapOperator class

2015-05-18 Thread asfgit
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] flink pull request: Wrap very long line in FlatMapOperator class

2015-05-18 Thread StephanEwen
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] flink pull request: Wrap very long line in FlatMapOperator class

2015-05-18 Thread fhueske
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] flink pull request: Wrap very long line in FlatMapOperator class

2015-05-17 Thread hsaputra
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] flink pull request: Wrap very long line in FlatMapOperator class

2015-05-15 Thread hsaputra
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