Mailing-List: contact dev-help@giraph.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@giraph.apache.org
Date: Wed, 1 Aug 2012 15:39:02 +0000 (UTC)
From: "Alessandro Presta (JIRA)" <jira@apache.org>
To: giraph-dev@incubator.apache.org
Message-ID: <1460773849.1219.1343835542920.JavaMail.jiratomcat@issues-vm>
In-Reply-To: <1538412782.59599.1342464515472.JavaMail.jiratomcat@issues-vm>
Subject: [jira] [Commented] (GIRAPH-259) TestBspBasic.testBspPageRank is
 broken
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/GIRAPH-259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13426702#comment-13426702 ] 

Alessandro Presta commented on GIRAPH-259:
------------------------------------------

Is there any particular reason why MasterCompute#compute() is executed before anything else? It seems to me that we would be more loyal to the BSP model if we didn't enforce this: MasterCompute is just a special vertex, executed during the superstep, whose vote to halt wins over everyone else's.
You still use it as the logical place where serial, graph-wide computations are done (for example, checking the global value of an aggregator as a termination condition), but you don't rely on the contract that it will be run at any specific time relative to normal vertices.

Especially since we are going to offer auto-resetting aggregators, it makes sense for MasterCompute to be a normal aggregator user, so collecting aggregators and running compute() are two orthogonal components.

@Avery: regarding those two failures, please see GIRAPH-276
                
> TestBspBasic.testBspPageRank is broken
> --------------------------------------
>
>                 Key: GIRAPH-259
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-259
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Maja Kabiljo
>            Assignee: Maja Kabiljo
>         Attachments: GIRAPH-259-1.patch, GIRAPH-259-2.patch, GIRAPH-259-3.patch
>
>
> Test crashes on line 152 in class SimplePageRankVertex in distributed mode.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira