Permalink
Browse files

Issue 223: changed default for client to use ThresholdFailureDetector…

… and added a comment to VoldemortConfig regarding need to change to default when issue 197 is resolved.
  • Loading branch information...
kirktrue committed Mar 1, 2010
1 parent 4fa61a9 commit e219ac64cfc2caf9ebbf4b3dea95a3c29d32202b
@@ -41,7 +41,7 @@
public class FailureDetectorConfig {
- public static final String DEFAULT_IMPLEMENTATION_CLASS_NAME = BannagePeriodFailureDetector.class.getName();
+ public static final String DEFAULT_IMPLEMENTATION_CLASS_NAME = ThresholdFailureDetector.class.getName();
public static final long DEFAULT_BANNAGE_PERIOD = 30000;
@@ -290,6 +290,7 @@ public VoldemortConfig(Props props) {
this.rebalancingServicePeriod = props.getInt("rebalancing.service.period.ms", 1000);
this.maxParallelStoresRebalancing = props.getInt("max.parallel.stores.rebalancing", 3);
+ // FIX: need to use ThresholdFailureDetector, but blocked on issue 197.
this.failureDetectorImplementation = props.getString("failuredetector.implementation",
BannagePeriodFailureDetector.class.getName());

0 comments on commit e219ac6

Please sign in to comment.