Github user jkbradley commented on the pull request:

    https://github.com/apache/spark/pull/10539#issuecomment-172128663
  
    @viirya About your above comments:
    
    > Routing tables should not be the problem here because the edges and 
vertices are from the same graph, so the routing table should be correct.
    
    I don't think that's true because ```normalized2``` is created using 
```Graph.apply```, which will recompute routing tables.
    
    I'd recommend using Graph.apply, and then exploring what PIC is doing in 
those 2 tests (e.g., by printing out the vertex and edge attributes on each 
iteration, and seeing how values are propagated around the graph).


---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to