Github user twalthr commented on the pull request:
https://github.com/apache/flink/pull/729#issuecomment-108640926
Hey Ufuk,
thank you very much for reviewing my code and all others for the feedback!
I tried to consider all your feedback (I hope I didn't forget anything). I did
a large refactoring again, added some comments to important parts of the code
and fixed some bugs. I also added some additional test cases. I hope the PR is
now ready to be merged (if the build succeeds) :)
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
