[jira] [Updated] (FLINK-3888) Custom Aggregator with Convergence can't be registered directly with DeltaIteration

2016-10-03 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3888?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri updated FLINK-3888: - Component/s: Iterations > Custom Aggregator with Convergence can't be registered direc

[jira] [Commented] (FLINK-3888) Custom Aggregator with Convergence can't be registered directly with DeltaIteration

2016-10-03 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15543133#comment-15543133 ] Vasia Kalavri commented on FLINK-3888: -- I had a quick look into this and I don't see any fundamental

[jira] [Commented] (FLINK-1815) Add methods to read and write a Graph as adjacency list

2016-09-30 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15536120#comment-15536120 ] Vasia Kalavri commented on FLINK-1815: -- The motivation for this issue was to (1) provide an easy way

[GitHub] flink issue #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-30 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2564 What I meant is simply creating an edge with a Tuple2 label containing the labels of the two edges in the bipartite graph. Makes sense? --- If your project is set up for it, you can reply

[jira] [Commented] (FLINK-1815) Add methods to read and write a Graph as adjacency list

2016-09-30 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15535748#comment-15535748 ] Vasia Kalavri commented on FLINK-1815: -- Hi [~fobeligi], representing a graph as an adjacency list

[GitHub] flink issue #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-30 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2564 Providing a flattened tuple is certainly better than having the user implement the reduce, but I think we should provide separate methods for the default and custom operations. A projection is a very

[GitHub] flink pull request #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-29 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2564#discussion_r81217922 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/BipartiteGraph.java --- @@ -0,0 +1,231 @@ +/* + * Licensed to the Apache

[GitHub] flink pull request #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-29 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2564#discussion_r81218538 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/Edge.java --- @@ -34,10 +34,10 @@ public Edge(){} - public

[GitHub] flink pull request #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-29 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2564#discussion_r81218403 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/BipartiteGraph.java --- @@ -0,0 +1,231 @@ +/* + * Licensed to the Apache

[GitHub] flink pull request #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-29 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2564#discussion_r81218056 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/BipartiteGraph.java --- @@ -0,0 +1,231 @@ +/* + * Licensed to the Apache

[GitHub] flink pull request #2564: [FLINK-2254] Add BipartiateGraph class

2016-09-29 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2564#discussion_r81218490 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/BipartiteGraph.java --- @@ -0,0 +1,231 @@ +/* + * Licensed to the Apache

[jira] [Commented] (FLINK-4646) Add BipartiteGraph class

2016-09-22 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4646?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15512880#comment-15512880 ] Vasia Kalavri commented on FLINK-4646: -- Hi [~StephanEwen], thanks! We've discussed this in the parent

[jira] [Commented] (FLINK-2254) Add Bipartite Graph Support for Gelly

2016-09-15 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15493427#comment-15493427 ] Vasia Kalavri commented on FLINK-2254: -- It sounds good to me [~ivan.mushketyk]. Please go ahead

[jira] [Commented] (FLINK-2254) Add Bipartite Graph Support for Gelly

2016-09-11 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15482143#comment-15482143 ] Vasia Kalavri commented on FLINK-2254: -- Hi [~ivan.mushketyk] thanks a lot for creating the design

[jira] [Commented] (FLINK-1707) Add an Affinity Propagation Library Method

2016-09-11 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15481915#comment-15481915 ] Vasia Kalavri commented on FLINK-1707: -- Hi [~joseprupi], I see that in order to create the graph you

[jira] [Commented] (FLINK-4522) Gelly link broken in homepage

2016-08-29 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4522?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15445732#comment-15445732 ] Vasia Kalavri commented on FLINK-4522: -- Hi Greg, what is the current redirect? The gelly link

[jira] [Commented] (FLINK-2254) Add Bipartite Graph Support for Gelly

2016-08-28 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15444180#comment-15444180 ] Vasia Kalavri commented on FLINK-2254: -- Hi [~ivan.mushketyk], what you are describing in "w

[jira] [Commented] (FLINK-1707) Add an Affinity Propagation Library Method

2016-08-28 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15443318#comment-15443318 ] Vasia Kalavri commented on FLINK-1707: -- Hi [~joseprupi], thanks for the update. Regarding

[jira] [Created] (FLINK-4522) Gelly link broken in homepage

2016-08-28 Thread Vasia Kalavri (JIRA)
Vasia Kalavri created FLINK-4522: Summary: Gelly link broken in homepage Key: FLINK-4522 URL: https://issues.apache.org/jira/browse/FLINK-4522 Project: Flink Issue Type: Bug

[jira] [Commented] (FLINK-4440) Make API for edge/vertex creation less verbose

2016-08-26 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4440?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15439178#comment-15439178 ] Vasia Kalavri commented on FLINK-4440: -- Alright, makes sense. Let's close this then. Is that fine

[jira] [Commented] (FLINK-4440) Make API for edge/vertex creation less verbose

2016-08-25 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4440?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15437307#comment-15437307 ] Vasia Kalavri commented on FLINK-4440: -- I don't have a strong opinion against adding the {{create

[GitHub] flink issue #2398: [gelly] Add Vertex.create and Edge.create helper methods

2016-08-21 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2398 Hi @mushketyk, thank you for this PR. Is there a corresponding JIRA for it? If not, can you please create one and tag the PR with it? Also, make sure you read the [How to contribute guide

[jira] [Issue Comment Deleted] (FLINK-2254) Add Bipartite Graph Support for Gelly

2016-08-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri updated FLINK-2254: - Comment: was deleted (was: Hi [~ivan.mushketyk], thank you for your interest. As far as I know

[jira] [Commented] (FLINK-2254) Add Bipartite Graph Support for Gelly

2016-08-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428076#comment-15428076 ] Vasia Kalavri commented on FLINK-2254: -- Hi [~ivan.mushketyk], thank you for your interest. As far

[jira] [Commented] (FLINK-2254) Add Bipartite Graph Support for Gelly

2016-08-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428075#comment-15428075 ] Vasia Kalavri commented on FLINK-2254: -- Hi [~ivan.mushketyk], thank you for your interest. As far

[jira] [Commented] (FLINK-1707) Add an Affinity Propagation Library Method

2016-08-15 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15421162#comment-15421162 ] Vasia Kalavri commented on FLINK-1707: -- Hi [~joseprupi], I haven't had time to look at your example

[GitHub] flink issue #2258: [FLINK-4104] [docs] Restructure Gelly docs

2016-08-10 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2258 Hi @greghogan, sorry I somehow missed this PR 😓 I really like the restructuring, thanks! I don't think it's a problem to rename to `gelly/index.html`. Just make sure to also update the link

[jira] [Commented] (FLINK-1707) Add an Affinity Propagation Library Method

2016-08-08 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15412194#comment-15412194 ] Vasia Kalavri commented on FLINK-1707: -- Thank you for the example [~joseprupi]. I'll try to get back

[jira] [Commented] (FLINK-4204) Clean up gelly-examples

2016-07-14 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15376834#comment-15376834 ] Vasia Kalavri commented on FLINK-4204: -- Separating drivers and examples sounds like a good idea. Do

[jira] [Commented] (FLINK-4204) Clean up gelly-examples

2016-07-12 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15373767#comment-15373767 ] Vasia Kalavri commented on FLINK-4204: -- [~greghogan] let me know what you think! > Clean up ge

[jira] [Created] (FLINK-4204) Clean up gelly-examples

2016-07-12 Thread Vasia Kalavri (JIRA)
Vasia Kalavri created FLINK-4204: Summary: Clean up gelly-examples Key: FLINK-4204 URL: https://issues.apache.org/jira/browse/FLINK-4204 Project: Flink Issue Type: Improvement

[GitHub] flink issue #2197: [FLINK-3916] [table] Allow generic types passing the Tabl...

2016-07-05 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2197 Thanks for the ping @twalthr! I don't see anything suspicious :) --- 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] flink issue #2032: [FLINK-3965] [gelly] Delegating GraphAlgorithm

2016-06-30 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2032 I've had an offline discussion with @greghogan about this. We won't be advertising `DelegatingGraphAlgorithm` as a user-facing feature. +1 from me! --- If your project is set up for it, you can reply

[GitHub] flink issue #2188: [FLINK-4135] [gelly] Replace ChecksumHashCode as GraphAna...

2016-06-30 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2188 Looks good, thanks! --- 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

[GitHub] flink issue #2184: [FLINK-3618] [gelly] Rename abstract UDF classes in Scatt...

2016-06-30 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2184 Thanks for the update @greghogan! +1 to 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

[jira] [Closed] (FLINK-2352) [Graph Visualization] Integrate Gelly with Gephi

2016-06-30 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2352?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri closed FLINK-2352. Resolution: Won't Fix I agree with Greg. Gephi doesn't seem like a good fit. > [Gr

[jira] [Commented] (FLINK-1759) Execution statistics for vertex-centric iterations

2016-06-30 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15357419#comment-15357419 ] Vasia Kalavri commented on FLINK-1759: -- I think it would be very helpful. A couple of users have

[GitHub] flink issue #2184: [FLINK-3618] [gelly] Rename abstract UDF classes in Scatt...

2016-06-29 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2184 Thanks a lot @greghogan. Everything looks good. Can you please update the docs too? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[jira] [Commented] (FLINK-1707) Add an Affinity Propagation Library Method

2016-06-29 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15355673#comment-15355673 ] Vasia Kalavri commented on FLINK-1707: -- Thanks [~joseprupi]. If you could document the example

[jira] [Commented] (FLINK-3879) Native implementation of HITS algorithm

2016-06-29 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15355664#comment-15355664 ] Vasia Kalavri commented on FLINK-3879: -- You and I are the only Gelly component "shep

[GitHub] flink issue #1967: [FLINK-3879] [gelly] Native implementation of HITS algori...

2016-06-28 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/1967 Hey @greghogan, was there consensus regarding this change? I see the numbers, but did anyone review this PR? I've been offline for the past few days, and I now see that nobody reviewed #2160

[jira] [Commented] (FLINK-3965) Delegating GraphAlgorithm

2016-06-23 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15346395#comment-15346395 ] Vasia Kalavri commented on FLINK-3965: -- The functionality is much needed in my opinion. My concern

[jira] [Commented] (FLINK-4101) Calculating average in Flink DataStream on window time

2016-06-23 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15346204#comment-15346204 ] Vasia Kalavri commented on FLINK-4101: -- Hi [~mrakki3110], JIRA is used for reporting bugs

[jira] [Closed] (FLINK-4101) Calculating average in Flink DataStream on window time

2016-06-23 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4101?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri closed FLINK-4101. Resolution: Invalid > Calculating average in Flink DataStream on window t

[jira] [Comment Edited] (FLINK-1514) [Gelly] Add a Gather-Sum-Apply iteration method

2016-06-23 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15346160#comment-15346160 ] Vasia Kalavri edited comment on FLINK-1514 at 6/23/16 9:27 AM: --- Hi [~michael

[jira] [Commented] (FLINK-1514) [Gelly] Add a Gather-Sum-Apply iteration method

2016-06-23 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15346160#comment-15346160 ] Vasia Kalavri commented on FLINK-1514: -- Hi [~michael], tis GSA implementation is a variation

[GitHub] flink issue #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2053 Hi @joseprupi, thanks a lot for this PR. I have looked through the code and I'm quite worried about its performance and scalability. Have you tried it out with any big inputs? We

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67873916 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67873432 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67873061 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67871432 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67869049 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67868962 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67868917 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink pull request #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2053#discussion_r67868911 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/AffinityPropagation.java --- @@ -0,0 +1,535 @@ +/* + * Licensed

[GitHub] flink issue #2053: [FLINK-1707] Affinity Propagation

2016-06-21 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2053 I'll shepherd this PR. --- 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

[jira] [Commented] (FLINK-1815) Add methods to read and write a Graph as adjacency list

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338596#comment-15338596 ] Vasia Kalavri commented on FLINK-1815: -- Hi [~fobeligi], are you by any chance still working

[jira] [Commented] (FLINK-3618) Rename abstract UDF classes in Scatter-Gather implementation

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338595#comment-15338595 ] Vasia Kalavri commented on FLINK-3618: -- [~greghogan] are you working on this? It would be nice

[jira] [Commented] (FLINK-3965) Delegating GraphAlgorithm

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338593#comment-15338593 ] Vasia Kalavri commented on FLINK-3965: -- Hey [~greghogan], do we want this in for 1.1.0? > Delegat

[jira] [Updated] (FLINK-2352) [Graph Visualization] Integrate Gelly with Gephi

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2352?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri updated FLINK-2352: - Assignee: (was: Shivani Ghatge) > [Graph Visualization] Integrate Gelly with Ge

[jira] [Commented] (FLINK-3879) Native implementation of HITS algorithm

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338592#comment-15338592 ] Vasia Kalavri commented on FLINK-3879: -- Hey [~greghogan], since FLINK-2044 was updated to provide

[jira] [Closed] (FLINK-2941) Implement a neo4j - Flink/Gelly connector

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2941?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri closed FLINK-2941. Resolution: Won't Fix The connector has been implemented by [~mju] in https://github.com/s1ck

[jira] [Commented] (FLINK-2941) Implement a neo4j - Flink/Gelly connector

2016-06-19 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2941?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338588#comment-15338588 ] Vasia Kalavri commented on FLINK-2941: -- Hi, is there any news regarding this? [~mju], are you going

[jira] [Commented] (FLINK-3916) Allow generic types passing the Table API

2016-06-15 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3916?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15332354#comment-15332354 ] Vasia Kalavri commented on FLINK-3916: -- FLINK-3615 is about extending or refactoring

[GitHub] flink issue #2032: [FLINK-3965] [gelly] Delegating GraphAlgorithm

2016-06-10 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/2032 Hi, I don't completely get the reasoning either, but it sounds interesting indeed! @greghogan, could you maybe clarify the following for me: - What do you mean by "mergeable" con

[GitHub] flink pull request: [FLINK-3806] [gelly] Revert use of DataSet.count()

2016-06-01 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/2036 Thanks @greghogan! It looks like you have some checkstyle violation, otherwise +1. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] flink pull request: Affinity Propagation

2016-05-31 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/2053#issuecomment-222640838 Hi @joseprupi, thanks for the PR! Just a heads-up. There is currently a very high number of unreviewed PRs and it might take a while until someone finds the time

[jira] [Commented] (FLINK-1707) Add an Affinity Propagation Library Method

2016-05-29 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15306054#comment-15306054 ] Vasia Kalavri commented on FLINK-1707: -- Thank you [~joseprupi]! When you are ready, open a PR and let

[jira] [Commented] (FLINK-1536) Graph partitioning operators for Gelly

2016-05-29 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-1536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15306053#comment-15306053 ] Vasia Kalavri commented on FLINK-1536: -- Hi [~rohit13k], a graph in gelly is represented by 2 datasets

[GitHub] flink pull request: [FLINK-3806] [gelly] Revert use of DataSet.cou...

2016-05-26 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/2036#issuecomment-221811872 Thanks for the PR @greghogan! I left one comment. The HITS implementation also makes use of the number of vertices in the same way PageRank did. We can update

[GitHub] flink pull request: [FLINK-3806] [gelly] Revert use of DataSet.cou...

2016-05-26 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2036#discussion_r64709459 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/gsa/GatherSumApplyIteration.java --- @@ -327,6 +343,11 @@ private SumUdf

[GitHub] flink pull request: [FLINK-3806] [gelly] Revert use of DataSet.cou...

2016-05-26 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2036#discussion_r64709426 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/gsa/GatherSumApplyIteration.java --- @@ -289,6 +300,11 @@ private GatherUdf

[jira] [Resolved] (FLINK-2044) Implementation of Gelly HITS Algorithm

2016-05-25 Thread Vasia Kalavri (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-2044?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasia Kalavri resolved FLINK-2044. -- Resolution: Implemented Fix Version/s: 1.1.0 > Implementation of Gelly HITS Algori

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-25 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-221637781 Thanks @gallenvara! All tests pass now. I 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

[GitHub] flink pull request: [FLINK-3945] [gelly] Degree annotation for dir...

2016-05-25 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2021#discussion_r64589442 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/asm/degree/annotate/directed/EdgeDegreesPair.java --- @@ -0,0 +1,81

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-25 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-221603967 The failures were in the `HITSAlgorithmITCase`. --- 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

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-25 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-221600182 Hey @gallenvara, I was about to merge this, but I see test failures after rebasing on top of master. Can you please (1) rebase on top of the latest master

[GitHub] flink pull request: [FLINK-3945] [gelly] Degree annotation for dir...

2016-05-25 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/2021#discussion_r64577865 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/asm/degree/annotate/directed/EdgeDegreesPair.java --- @@ -0,0 +1,81

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-25 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-221504600 Thank you @gallenvara. I'll merge later today. --- 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

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-24 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-221394386 Hey @gallenvara, I had a private chat with @greghogan about this PR. We think that we should change the label type to a boolean instead of string. It should make

[GitHub] flink pull request: [FLINK-3728] [tableAPI] Improve error message ...

2016-05-24 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/2018#issuecomment-221185294 Also looks good! +1. --- 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

[GitHub] flink pull request: [FLINK-3955] [tableAPI] Rename Table.toSink() ...

2016-05-24 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/2023#issuecomment-221184374 +1 looks good! --- 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

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-20 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1956#issuecomment-220628229 Thank you for the update @gallenvara and for your patience with our continuous comments :) How many edges did the graphs in your experiment have? --- If your project

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1956#discussion_r64054001 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/HITSAlgorithm.java --- @@ -273,4 +289,23 @@ public void sendMessages(Vertex

[GitHub] flink pull request: [FLINK-2044] [gelly] Implementation of Gelly H...

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1956#discussion_r64051693 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/HITSAlgorithm.java --- @@ -243,18 +255,22 @@ public void sendMessages

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r64049343 --- Diff: docs/apis/batch/libs/gelly.md --- @@ -2055,22 +2055,22 @@ vertex and edge in the output graph stores the common group value and the number

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-20 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1980#issuecomment-220620841 Thanks for the update @greghogan. I left a few comments for improvements of the docs. After these are addressed it should be good to merge. --- If your project is set up

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r64048955 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/similarity/JaccardIndex.java --- @@ -43,11 +43,13 @@ import

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r64048499 --- Diff: flink-libraries/flink-gelly-examples/src/main/java/org/apache/flink/graph/examples/JaccardIndex.java --- @@ -54,73 +57,112 @@ public

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r64048412 --- Diff: flink-libraries/flink-gelly-examples/src/main/java/org/apache/flink/graph/examples/JaccardIndex.java --- @@ -40,9 +43,9 @@ /** * Driver

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-20 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r64048334 --- Diff: docs/apis/batch/libs/gelly.md --- @@ -2250,14 +2250,33 @@ graph.run(new TranslateVertexValues(new LongValueAddOffset(vertexCount

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63899403 --- Diff: flink-libraries/flink-gelly/src/test/java/org/apache/flink/graph/library/similarity/JaccardIndexTest.java --- @@ -0,0 +1,136

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63896459 --- Diff: flink-libraries/flink-gelly/src/test/java/org/apache/flink/graph/library/similarity/JaccardIndexTest.java --- @@ -0,0 +1,136

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63895157 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/similarity/JaccardIndex.java --- @@ -0,0 +1,462 @@ +/* + * Licensed

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63894471 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/similarity/JaccardIndex.java --- @@ -0,0 +1,462 @@ +/* + * Licensed

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63894348 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/similarity/JaccardIndex.java --- @@ -0,0 +1,462 @@ +/* + * Licensed

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63894239 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/similarity/JaccardIndex.java --- @@ -0,0 +1,462 @@ +/* + * Licensed

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63894069 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/library/similarity/JaccardIndex.java --- @@ -0,0 +1,462 @@ +/* + * Licensed

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63893564 --- Diff: docs/apis/batch/libs/gelly.md --- @@ -2051,6 +2052,26 @@ The algorithm takes a directed, vertex (and possibly edge) attributed graph as i vertex

[GitHub] flink pull request: [FLINK-3780] [gelly] Jaccard Similarity

2016-05-19 Thread vasia
Github user vasia commented on a diff in the pull request: https://github.com/apache/flink/pull/1980#discussion_r63893177 --- Diff: flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/asm/degree/annotate/TranslateEdgeDegreeToIntValue.java --- @@ -0,0 +1,51

<    1   2   3   4   5   6   7   8   9   10   >