[GitHub] flink pull request: FLINK-2949 [WIP]: Add method 'writeSequencefil...

2015-12-26 Thread smarthi
Github user smarthi commented on the pull request: https://github.com/apache/flink/pull/1311#issuecomment-167371893 I was gonna close this Jira and withdraw the PR. Closing now.. --- 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: FLINK-2949 [WIP]: Add method 'writeSequencefil...

2015-12-26 Thread smarthi
Github user smarthi closed the pull request at: https://github.com/apache/flink/pull/1311 --- 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 ena

[GitHub] flink pull request: FLINK-2949 [WIP]: Add method 'writeSequencefil...

2015-12-26 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1311#issuecomment-167370890 I would like to not add this to DataSet. Nick said it well in the discussion of another issue: This gives nice syntactic sugar, looks super in examples and on slides

[GitHub] flink pull request: FLINK-2949 [WIP]: Add method 'writeSequencefil...

2015-12-01 Thread smarthi
Github user smarthi commented on the pull request: https://github.com/apache/flink/pull/1311#issuecomment-161066615 @rmetzger This fell off my radar briefly as other things took priority. Let me know what you think about adding this to DataSet API, this follows from a similar change t

[GitHub] flink pull request: FLINK-2949 [WIP]: Add method 'writeSequencefil...

2015-12-01 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/1311#issuecomment-161058586 Thank you for the contribution. To me it seems that the pull request so far consists only of unrelated changes. Did you make any progress on resolving the issue?

[GitHub] flink pull request: FLINK-2949 [WIP]: Add method 'writeSequencefil...

2015-10-30 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/flink/pull/1311 FLINK-2949 [WIP]: Add method 'writeSequencefile' to DataSet You can merge this pull request into a Git repository by running: $ git pull https://github.com/smarthi/flink Flink-2949 Alternativ