Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/505
---
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 user vasia commented on the pull request:
https://github.com/apache/flink/pull/505#issuecomment-86449745
Thank you @andralungu! Will merge later.
---
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
Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/505#issuecomment-86366858
Travis passed :D
---
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 feat
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/505#issuecomment-86007328
Thanks a lot for the updates @andralungu and for testing this on a cluster
environment :-)
The example code looks much nicer now ^-^.
Only minor issue is that the f
Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/505#issuecomment-85725549
@vasia ,
I integrated your suggestions, the example works on the cluster for Orkut,
max iter = 20.
---
If your project is set up for it, you can reply to thi
Github user vasia commented on a diff in the pull request:
https://github.com/apache/flink/pull/505#discussion_r27017460
--- Diff:
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/library/SimpleCommunityDetection.java
---
@@ -0,0 +1,151 @@
+/*
+ * Licensed to
Github user vasia commented on a diff in the pull request:
https://github.com/apache/flink/pull/505#discussion_r27017296
--- Diff:
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/library/SimpleCommunityDetection.java
---
@@ -0,0 +1,151 @@
+/*
+ * Licensed to
Github user vasia commented on a diff in the pull request:
https://github.com/apache/flink/pull/505#discussion_r27017030
--- Diff:
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/SimpleCommunityDetectionExample.java
---
@@ -0,0 +1,159 @@
+/*
+ * Lice
Github user vasia commented on a diff in the pull request:
https://github.com/apache/flink/pull/505#discussion_r27016738
--- Diff:
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/SimpleCommunityDetectionExample.java
---
@@ -0,0 +1,159 @@
+/*
+ * Lice
Github user vasia commented on a diff in the pull request:
https://github.com/apache/flink/pull/505#discussion_r27016590
--- Diff:
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/SimpleCommunityDetectionExample.java
---
@@ -0,0 +1,159 @@
+/*
+ * Lice
Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/505#issuecomment-85194124
Leave it to @rmetzger to be uber meticulous :D
I will add the bullet point! Thanks!
---
If your project is set up for it, you can reply to this email and have your
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/505#issuecomment-85173349
I didn't see a bigger issue in the code, but @vasia is the authority on
this ;)
It would be nice to add at least another bullet to this list:
http://ci.apache.org/pro
GitHub user andralungu opened a pull request:
https://github.com/apache/flink/pull/505
[FLINK-1726][gelly] Added Community Detection Library and Example
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/andralungu/flink flink_commu
13 matches
Mail list logo