[GitHub] flink pull request: Hits

2015-11-13 Thread mfahimazizi
Github user mfahimazizi closed the pull request at: https://github.com/apache/flink/pull/765 --- 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

[GitHub] flink pull request: Hits

2015-10-19 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-149148599 Hi @mfahimazizi, is there any update regarding this PR? If you're not planning to keep working on it, could you please close this? Thanks a lot! --- If your project

[GitHub] flink pull request: Hits

2015-09-16 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-140669850 Hey @mfahimazizi, thank you for your reply! Could you give me some details on what you mean by "calculate edges values"? What is the computation that you want to do and

[GitHub] flink pull request: Hits

2015-09-16 Thread mfahimazizi
Github user mfahimazizi commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-140871748 Hey @vasia, Thank you. sorry it is "calculate node values". first we label the edge as 'H' (hub score), then undirected the graph and label the new edges as

[GitHub] flink pull request: Hits

2015-09-16 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-140903149 Thanks for the explanation @mfahimazizi. So, if you put the edge label inside the message that you send in the messaging function, then you will know which values to

[GitHub] flink pull request: Hits

2015-09-15 Thread mfahimazizi
Github user mfahimazizi commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-140633835 Hi @vasia, Thank you. for this algorithm, we need to calculate edges values inside VertexUpdateFunction, so I will try to build my own delta iteration. --- If

[GitHub] flink pull request: Hits

2015-09-15 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-140314739 Hi @mfahimazizi, as I said, it is not possible to access the edge values inside the `VertexUpdateFunction`. However, you can get the same functionality if you

[GitHub] flink pull request: Hits

2015-09-14 Thread mfahimazizi
Github user mfahimazizi commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-140280883 @andralungu and @vasia , we are outside germany :( we need to access edges inside vertexupdate function (not inside messaging function), as we mentioned in above

[GitHub] flink pull request: Hits

2015-08-11 Thread andralungu
Github user andralungu commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-130016029 @vasia , I guess the guys already finished the IMPRO3 course. What is the status of this PR @mfahimazizi? Do you have time to address the comments? Are you

[GitHub] flink pull request: Hits

2015-07-07 Thread mfahimazizi
Github user mfahimazizi commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-119188834 Hi @vasia, sorry for late reply. I set indentation with tabs instead of spaces in IDE. and push new updated files. as I discuss in jira issue about

[GitHub] flink pull request: Hits

2015-07-02 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-118161158 Hi there! What is the status of this PR? It seems that travis is failing with checkstyle violations related to using spaces instead of tabs. Let us know if you

[GitHub] flink pull request: Hits

2015-06-03 Thread andralungu
Github user andralungu commented on a diff in the pull request: https://github.com/apache/flink/pull/765#discussion_r31677964 --- Diff: flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/library/HITS.java --- @@ -0,0 +1,180 @@ +/* + * Licensed to the Apache

[GitHub] flink pull request: Hits

2015-06-03 Thread andralungu
Github user andralungu commented on the pull request: https://github.com/apache/flink/pull/765#issuecomment-108634942 Hi @mfahimazizi, The common practice is to prefix the commit with the Jira issue, e.g. [FLINK-20XX][gelly]. Also, you should play with your IDE's settings.

[GitHub] flink pull request: Hits

2015-06-03 Thread andralungu
Github user andralungu commented on a diff in the pull request: https://github.com/apache/flink/pull/765#discussion_r31678387 --- Diff: flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/HITSExample.java --- @@ -0,0 +1,131 @@ +/* + * Licensed to the

[GitHub] flink pull request: Hits

2015-06-03 Thread andralungu
Github user andralungu commented on a diff in the pull request: https://github.com/apache/flink/pull/765#discussion_r31678355 --- Diff: flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/HITSExample.java --- @@ -0,0 +1,131 @@ +/* + * Licensed to the

[GitHub] flink pull request: Hits

2015-06-03 Thread hsaputra
Github user hsaputra commented on a diff in the pull request: https://github.com/apache/flink/pull/765#discussion_r31596533 --- Diff: flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/utils/Hits.java --- @@ -0,0 +1,14 @@ +package org.apache.flink.graph.utils;

[GitHub] flink pull request: Hits

2015-06-03 Thread hsaputra
Github user hsaputra commented on a diff in the pull request: https://github.com/apache/flink/pull/765#discussion_r31596508 --- Diff: flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/HITSExample.java --- @@ -0,0 +1,113 @@ +package

[GitHub] flink pull request: Hits

2015-06-02 Thread mfahimazizi
GitHub user mfahimazizi opened a pull request: https://github.com/apache/flink/pull/765 Hits HITS algorithm in Gelly. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mfahimazizi/flink HITS Alternatively you can review and apply

[GitHub] flink pull request: Hits

2015-05-19 Thread JavidMayar
GitHub user JavidMayar opened a pull request: https://github.com/apache/flink/pull/694 Hits You can merge this pull request into a Git repository by running: $ git pull https://github.com/JavidMayar/flink HITS Alternatively you can review and apply these changes as the patch

[GitHub] flink pull request: Hits

2015-05-19 Thread JavidMayar
Github user JavidMayar closed the pull request at: https://github.com/apache/flink/pull/694 --- 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