Github user PieterJanVanAeken commented on the pull request:
https://github.com/apache/flink/pull/808#issuecomment-114468380
Hi,
Thanks all for the comments. Unfortunately, I will be gone until the 13th
of July so I will not be able to resolve these issues until then. They are
however all valid points. One more note perhaps, in regard to the unit tests, I
added the basic ones (based on the Java version of them), but they are most
certainly not all there. The same can be said for the methods that were added
recently to the java Gelly API.
Since I am a bit strapped for time atm, I was hoping to push this into the
official Flink repo (maybe not master branch?) so other people could more
easily contribute and help resolve these final issues thus speeding up the
process a bit.
---
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.
---
