[ 
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)

Reply via email to