Github user mbalassi commented on the pull request:
https://github.com/apache/flink/pull/573#issuecomment-90132428
Hey Matthias,
Thanks for opening the pull request as discussed on the mailing list. Could
you please provide some description here also?
Please do another update on your PR to better match our coding guidelines:
1. The first six commits are not needed. Th first one is already merged,
than you have 5 huge merge commits, that we are trying to avoid.
2. Some of your commit messages need a bit more polishing, the "added raw
output type support" and "added API compatibility classes" do not describe your
work when merged to the Flink code. I would also suggest to add a [streaming]
tag for your commits.
3. Please squash 605de53 to 1519d36, becuase it is just a minor fix for
that.
---
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.
---
