Mailing-List: contact dev-help@giraph.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@giraph.apache.org
Date: Wed, 9 Jan 2013 20:28:13 +0000 (UTC)
From: "Eli Reisman (JIRA)" <jira@apache.org>
To: giraph-dev@incubator.apache.org
Message-ID: <JIRA.12626124.1357333110415.108708.1357763293304@arcas>
In-Reply-To: <JIRA.12626124.1357333110415@arcas>
References: <JIRA.12626124.1357333110415@arcas>
Subject: [jira] [Commented] (GIRAPH-469) Cleanup GraphMapper
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Eli Reisman commented on GIRAPH-469:
------------------------------------

Also: nitay, i could not tell from GIRAPH-409 if you had already done this or not, but was working on the refactor of GraphMapper myself when I started uploading all these patches. If you had something in mind for this already, don't let me crowd you out, please post your patch! Sorry in advance if thats what happened ;)

                
> Cleanup GraphMapper
> -------------------
>
>                 Key: GIRAPH-469
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-469
>             Project: Giraph
>          Issue Type: Improvement
>            Reporter: Nitay Joffe
>            Assignee: Eli Reisman
>         Attachments: GIRAPH-469-1-eli-idea.patch, GIRAPH-469-2.patch
>
>
> I don't see why we even call a map() method seeing as we are overriding run(). We are clearly not particularly "mapreduce-y" so we should make it our entry point more clear than a map(). Also I think we should have something like a WorkerThread similar to MasterThread and clean up all of this to just creare whichever threads the node is assigned roles of. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira