Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/408#issuecomment-77341085
Not sure why the last Travis check failed because the pom.xml seems to be
the correct one.
But apart from that, this looks very nice and compliant with the vertex
centric approach.
However, there is a bug there, in runVertexCentricIteration: the superstep
number does not change when you run tests in Collection mode and I have a hunch
that it would also be reproducible here.
Apart from that. Good to merge from my side.
And as future work, once I finish the vertexCentricIteration extensions
which will allow the user to choose the messaging direction, we should exend
GSA to do the same :D
---
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.
---
