[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-05-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/596 --- 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 enabl

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-05-01 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/596#issuecomment-98166735 :+1: for this! Will merge. --- 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

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-04-20 Thread andralungu
Github user andralungu commented on the pull request: https://github.com/apache/flink/pull/596#issuecomment-94438358 @vasia , The library method matches Spargel now. I also refactored the randomised edges test a bit. Tell me what you think! --- If your project is set up

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-04-18 Thread andralungu
Github user andralungu commented on the pull request: https://github.com/apache/flink/pull/596#issuecomment-94177583 Hi @vasia , I will change the code to match Spargel, probably Monday :) I came across this class in our tests the other day, while searching for something e

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-04-18 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/596#discussion_r28645013 --- Diff: flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/library/ConnectedComponents.java --- @@ -0,0 +1,90 @@ +/* + * Licensed to the A

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-04-14 Thread andralungu
Github user andralungu commented on the pull request: https://github.com/apache/flink/pull/596#issuecomment-92750207 Hi @vasia , I wanted to be as close to the example in the guide, to show that Spargel code can easily be ported to Gelly, hence the name. I changed it now :)

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-04-13 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/596#issuecomment-92494915 Hey @andralungu! This is actually the connected components algorithm. You can rename it to that. And take a look at the corresponding [Spargel example](https://githu

[GitHub] flink pull request: [FLINK-1883][gelly] Added Min Vertex Id Propag...

2015-04-13 Thread andralungu
GitHub user andralungu opened a pull request: https://github.com/apache/flink/pull/596 [FLINK-1883][gelly] Added Min Vertex Id Propagation library and example You can merge this pull request into a Git repository by running: $ git pull https://github.com/andralungu/flink minID