Github user chiwanpark commented on the pull request:
https://github.com/apache/flink/pull/1486#issuecomment-169283768
Hi @ajaybhat, thanks for opening pull request.
I read your pull request quickly and have a comment.
Currently, Flink has a lot of tests that create a temporary file and write
temporary result to the file. It makes time to test slow. So Flink community
decided to change these tests to avoid temporary file.
Could you re-implement `DummyRecordWriter` to make tests fast?
By the way, the author email of your commit (be59077) seems different email
address of your github account. Is this intentional?
---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
