Github user aljoscha commented on the pull request:
https://github.com/apache/flink/pull/977#issuecomment-128131074
Hi,
unfortunately the PR still needs a bit of work: The Class documentation
does not match the class. It would be good if the ports where not statically
set since this might interfere with concurrently running jobs. The code could
benefit from a bit of commenting. For example, I don't know why we need to
tests, for invoke and without invoke.
Maybe someone else can take a look? I'm on vacation right now and don't
really have a lot of time.
---
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.
---
