You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@giraph.apache.org by ac...@apache.org on 2012/02/25 09:18:00 UTC
svn commit: r1293545 - in /incubator/giraph/trunk: CHANGELOG
src/main/java/org/apache/giraph/graph/BspService.java
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;
}
/**