incubator-giraph-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ach...@apache.org
Subject svn commit: r1293545 - in /incubator/giraph/trunk: CHANGELOG src/main/java/org/apache/giraph/graph/BspService.java
Date Sat, 25 Feb 2012 08:18:00 GMT
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;
   }
 
   /**



Mime
View raw message