Github user MohamedNadjibMAMI commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-94259921
"If you want you can write a blog post about using Parquet with Flink."
This would a great plus for the project. Hope it goes a bit deeper and enters
the officia
Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/287
---
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 fhueske commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-70112509
LGTM
Will do some tests, clean-up and merge if everything is fine
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user FelixNeutatz commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-69513699
I rebased some of the commits as requested
---
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 pro
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-69358588
Yes, that would be nice. The test looks good.
A bit of style cleanup and some commit squashing and this is good to merge
in my opinion.
---
If your project i
Github user FelixNeutatz commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-69222057
Shall also add the same test for the mapred package?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-69221721
That looks good, thank you!
---
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 ha
Github user FelixNeutatz commented on the pull request:
https://github.com/apache/flink/pull/287#issuecomment-69220758
I added a simple unit test: HadoopInputFormatTest.java
Did you mean this test?
---
If your project is set up for it, you can reply to this email and have your
rep
