[ https://issues.apache.org/jira/browse/FLINK-2044?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15289658#comment-15289658 ]
ASF GitHub Bot commented on FLINK-2044: --------------------------------------- Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-220134968 Thanks for the update @gallenvara and @greghogan for the review. I have left a few comments in places where I couldn't figure out the code logic. There is one more thing I'd like to discuss with you. The current implementation is using `EdgeDirection.ALL`, but this feature is not very efficiently implemented. It actually performs 2 coGroups (one on the source Id and one on the target Id) and unions the outputs to create the result of the messaging function. Instead, I was thinking we could do the following: mark the inputs graph edges with an "authority" label and add opposite-direction edges with a "hub" authority. Then, we can use the default edge direction (IN) to perform the iteration. In even supersteps, each vertex sends messages only along "authority" edges and in odd supersteps only along "hub" edges. Does this make sense? > Implementation of Gelly HITS Algorithm > -------------------------------------- > > Key: FLINK-2044 > URL: https://issues.apache.org/jira/browse/FLINK-2044 > Project: Flink > Issue Type: New Feature > Components: Gelly > Reporter: Ahamd Javid > Assignee: GaoLun > Priority: Minor > > Implementation of Hits Algorithm in Gelly API using Java. the feature branch > can be found here: (https://github.com/JavidMayar/flink/commits/HITS) -- This message was sent by Atlassian JIRA (v6.3.4#6332)