Return-Path: X-Original-To: apmail-giraph-dev-archive@www.apache.org Delivered-To: apmail-giraph-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 299A917C2A for ; Fri, 31 Oct 2014 08:47:34 +0000 (UTC) Received: (qmail 32480 invoked by uid 500); 31 Oct 2014 08:47:34 -0000 Delivered-To: apmail-giraph-dev-archive@giraph.apache.org Received: (qmail 32433 invoked by uid 500); 31 Oct 2014 08:47:33 -0000 Mailing-List: contact dev-help@giraph.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@giraph.apache.org Delivered-To: mailing list dev@giraph.apache.org Received: (qmail 32341 invoked by uid 500); 31 Oct 2014 08:47:33 -0000 Delivered-To: apmail-incubator-giraph-dev@incubator.apache.org Received: (qmail 32338 invoked by uid 99); 31 Oct 2014 08:47:33 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 31 Oct 2014 08:47:33 +0000 Date: Fri, 31 Oct 2014 08:47:33 +0000 (UTC) From: "Martin Junghanns (JIRA)" To: giraph-dev@incubator.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (GIRAPH-962) TextAggregatorWriter with frequency AT_THE_END writes in every superstep MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/GIRAPH-962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Junghanns updated GIRAPH-962: ------------------------------------ Attachment: GIRAPH-962.patch Modified the condition as stated in the issue documentation. Tested with my own algorithms and with RandomWalkWithRestartComputationTest. > TextAggregatorWriter with frequency AT_THE_END writes in every superstep > ------------------------------------------------------------------------ > > Key: GIRAPH-962 > URL: https://issues.apache.org/jira/browse/GIRAPH-962 > Project: Giraph > Issue Type: Bug > Components: bsp > Affects Versions: 1.1.0 > Reporter: Martin Junghanns > Priority: Minor > Labels: aggregator, patch > Fix For: 1.1.0 > > Attachments: GIRAPH-962.patch > > Original Estimate: 0.5h > Remaining Estimate: 0.5h > > TextAggregatorWriter is used to write the content of all aggregators in a superstep to a text file. It offers three predefined frequencies: > {{NEVER (0)}} > {{ALWAYS (1) // in each superstep}} > {{AT_THE_END (-1) // only in the last superstep}} > The aggregators are written in a superstep if the following condition evaluates to true: > {{(frequeny == AT_THE_END && superstep == LAST_SUPERSTEP) || (frequency != NEVER && superstep % frequency == 0)}} > The problem is, that {{LAST_SUPERSTEP}} is represented by {{-1}} and {{x % -1 = 0}} for all {{x}}. This leads to the case, that if {{frequency = AT_THE_END}}, the content is written in every superstep. > On way to solve this is to extend the second part of the OR-clause to {{(frequency != NEVER && frequency != AT_THE_END && superstep % frequency == 0)}} -- This message was sent by Atlassian JIRA (v6.3.4#6332)