Sorry I ended up out of the loop this week, sounds good to me too!
On Wed, Sep 19, 2012 at 4:25 PM, Avery Ching (JIRA) <j...@apache.org> wrote: > > [ > https://issues.apache.org/jira/browse/GIRAPH-293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13459210#comment-13459210] > > Avery Ching commented on GIRAPH-293: > ------------------------------------ > > +1. This is Any objections Eli? > > > Should aggregators be checkpointed? > > ----------------------------------- > > > > Key: GIRAPH-293 > > URL: https://issues.apache.org/jira/browse/GIRAPH-293 > > Project: Giraph > > Issue Type: Bug > > Reporter: Alessandro Presta > > Assignee: Maja Kabiljo > > Attachments: GIRAPH-293.patch, GIRAPH-293.patch, > GIRAPH-293.patch, GIRAPH-293.patch > > > > > > As I understand, we don't include aggregators in checkpoints because > they are kept in the Zookeeper. > > One of our bootcampers is working on fixing TestManualCheckpoint, which > currently involves starting a new job from a checkpoint from a previous > job*. > > If this is a functionality we want going forward, then persistent > aggregators should be checkpointed. > > [*] That test relies on the fact that either aggregators are > checkpointed or they are always reset at each superstep. None of these is > happening, but the error cancels out with the fact that we are not actually > resuming from a checkpoint, but re-running the job from scratch. > > -- > 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 >