giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Maja Kabiljo (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (GIRAPH-355) Partition.readFields crashes
Date Tue, 02 Oct 2012 15:35:08 GMT

     [ https://issues.apache.org/jira/browse/GIRAPH-355?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Maja Kabiljo updated GIRAPH-355:
--------------------------------

    Attachment: GIRAPH-355.patch

Here is the patch, there were quite a few places where Context needed to be added so we would
always have it for Partition.

Crash happens when you run TestAutoCheckpoint in pseudo distributed mode. Or you can add one
line to TestManualCheckpoint to set the superstep which we want to restart from:
restartedJob.getConfiguration().setLong(GiraphConfiguration.RESTART_SUPERSTEP, 4);
This test will end up crashing even when you remove progress calls because GIRAPH-293 is not
in yet, but this null exception happens first.
                
> Partition.readFields crashes
> ----------------------------
>
>                 Key: GIRAPH-355
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-355
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Maja Kabiljo
>            Assignee: Maja Kabiljo
>         Attachments: GIRAPH-355.patch
>
>
> Partition:157 (vertex.getContext().progress();) will crash since graphState for vertex
is null at this point. We need to get Context inside Partition in order to be able to call
progress here. 
> The reason why this wasn't noticed is that our local tests don't try checkpoint restart
at the moment, and that's the only place where Partition.readFields is called.

--
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

Mime
View raw message