Author: aching Date: Sat Feb 25 08:18:00 2012 New Revision: 1293545 URL: http://svn.apache.org/viewvc?rev=1293545&view=rev Log: GIRAPH-87: Simplify boolean expression in BspService::checkpointFrequencyMet (Eli Reisman via aching).
Modified: incubator/giraph/trunk/CHANGELOG incubator/giraph/trunk/src/main/java/org/apache/giraph/graph/BspService.java Modified: incubator/giraph/trunk/CHANGELOG URL: http://svn.apache.org/viewvc/incubator/giraph/trunk/CHANGELOG?rev=1293545&r1=1293544&r2=1293545&view=diff ============================================================================== --- incubator/giraph/trunk/CHANGELOG (original) +++ incubator/giraph/trunk/CHANGELOG Sat Feb 25 08:18:00 2012 @@ -2,6 +2,9 @@ Giraph Change Log Release 0.2.0 - unreleased + GIRAPH-87: Simplify boolean expression in + BspService::checkpointFrequencyMet (Eli Reisman via aching). + GIRAPH-150: PageRankBenchmark accesses wrong conf after GiraphJob is created (aching). Modified: incubator/giraph/trunk/src/main/java/org/apache/giraph/graph/BspService.java URL: http://svn.apache.org/viewvc/incubator/giraph/trunk/src/main/java/org/apache/giraph/graph/BspService.java?rev=1293545&r1=1293544&r2=1293545&view=diff ============================================================================== --- incubator/giraph/trunk/src/main/java/org/apache/giraph/graph/BspService.java (original) +++ incubator/giraph/trunk/src/main/java/org/apache/giraph/graph/BspService.java Sat Feb 25 08:18:00 2012 @@ -584,10 +584,8 @@ public abstract class BspService<I exten } if (superstep < firstCheckpoint) { return false; - } else if (((superstep - firstCheckpoint) % checkpointFrequency) == 0) { - return true; } - return false; + return ((superstep - firstCheckpoint) % checkpointFrequency) == 0; } /**