Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/537#issuecomment-94915061
Hi @vasia ,
I added some answers to your inline comments! I will push my latest version
for this tomorrow.
Regarding the suggestion that the Vertex class might not be the best place
for the getDegree methods, there is a reason why implementing this took a while
^^. I wanted to make the degrees available only in the iteration. The problem
is that with the current code(this one and the one in "production"), a vertex
is only accessible in the updateVertex() method. This means that there is no
way to get the vertex within the iteration, you don't have a Vertex object,
it's not something you send to the class. That was the quick workaround I found
for this issue. If you know a better way, I am eager to hear your suggestions.
Thanks! :)
---
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.
---
