[GitHub] flink pull request: [FLINK-2377] Add reader.close() to readAllResu...

2015-07-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/924 --- 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

[GitHub] flink pull request: [FLINK-2377] Add reader.close() to readAllResu...

2015-07-21 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/924#issuecomment-123326390 +1, will merge this... --- 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

[GitHub] flink pull request: [FLINK-2377] Add reader.close() to readAllResu...

2015-07-20 Thread ggevay
GitHub user ggevay opened a pull request: https://github.com/apache/flink/pull/924 [FLINK-2377] Add reader.close() to readAllResultLines You can merge this pull request into a Git repository by running: $ git pull https://github.com/ggevay/flink readAllResultLinesFix

[GitHub] flink pull request: [FLINK-2377] Add reader.close() to readAllResu...

2015-07-20 Thread chiwanpark
Github user chiwanpark commented on the pull request: https://github.com/apache/flink/pull/924#issuecomment-122893574 Great catch! Looks 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