X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2FDefaultConfigParamsImpl.java;h=f5f410c75b5f6117bd1f3fea6371d521a971517a;hp=a2092234d54134fbe3de765d17041ad766d689c4;hb=b8c6400766f7324dd57d059bd48e435569fe1a27;hpb=274cee29fd987e5218f00a0352a1506adc2736ed diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/DefaultConfigParamsImpl.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/DefaultConfigParamsImpl.java index a2092234d5..f5f410c75b 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/DefaultConfigParamsImpl.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/DefaultConfigParamsImpl.java @@ -7,9 +7,17 @@ */ package org.opendaylight.controller.cluster.raft; -import scala.concurrent.duration.FiniteDuration; - +import com.google.common.base.Preconditions; +import com.google.common.base.Strings; +import com.google.common.base.Supplier; +import com.google.common.base.Suppliers; import java.util.concurrent.TimeUnit; +import javax.annotation.Nonnull; +import org.opendaylight.controller.cluster.raft.policy.DefaultRaftPolicy; +import org.opendaylight.controller.cluster.raft.policy.RaftPolicy; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; +import scala.concurrent.duration.FiniteDuration; /** * Default implementation of the ConfigParams @@ -18,6 +26,8 @@ import java.util.concurrent.TimeUnit; */ public class DefaultConfigParamsImpl implements ConfigParams { + private static final Logger LOG = LoggerFactory.getLogger(DefaultConfigParamsImpl.class); + private static final int SNAPSHOT_BATCH_COUNT = 20000; private static final int JOURNAL_RECOVERY_LOG_BATCH_SIZE = 1000; @@ -40,27 +50,62 @@ public class DefaultConfigParamsImpl implements ConfigParams { public static final FiniteDuration HEART_BEAT_INTERVAL = new FiniteDuration(100, TimeUnit.MILLISECONDS); - private FiniteDuration heartBeatInterval = HEART_BEAT_INTERVAL; private long snapshotBatchCount = SNAPSHOT_BATCH_COUNT; private int journalRecoveryLogBatchSize = JOURNAL_RECOVERY_LOG_BATCH_SIZE; - private FiniteDuration isolatedLeaderCheckInterval = - new FiniteDuration(HEART_BEAT_INTERVAL.length() * 1000, HEART_BEAT_INTERVAL.unit()); + private long isolatedLeaderCheckInterval = HEART_BEAT_INTERVAL.$times(1000).toMillis(); + private FiniteDuration electionTimeOutInterval; + + // 12 is just an arbitrary percentage. This is the amount of the total memory that a raft actor's + // in-memory journal can use before it needs to snapshot + private int snapshotDataThresholdPercentage = 12; + + private int snapshotChunkSize = SNAPSHOT_CHUNK_SIZE; + + private long electionTimeoutFactor = 2; + private String customRaftPolicyImplementationClass; + + private final Supplier policySupplier = Suppliers.memoize(new PolicySupplier()); + + private PeerAddressResolver peerAddressResolver = NoopPeerAddressResolver.INSTANCE; public void setHeartBeatInterval(FiniteDuration heartBeatInterval) { this.heartBeatInterval = heartBeatInterval; + electionTimeOutInterval = null; } public void setSnapshotBatchCount(long snapshotBatchCount) { this.snapshotBatchCount = snapshotBatchCount; } + public void setSnapshotDataThresholdPercentage(int snapshotDataThresholdPercentage){ + this.snapshotDataThresholdPercentage = snapshotDataThresholdPercentage; + } + + public void setSnapshotChunkSize(int snapshotChunkSize) { + this.snapshotChunkSize = snapshotChunkSize; + } + public void setJournalRecoveryLogBatchSize(int journalRecoveryLogBatchSize) { this.journalRecoveryLogBatchSize = journalRecoveryLogBatchSize; } public void setIsolatedLeaderCheckInterval(FiniteDuration isolatedLeaderCheckInterval) { - this.isolatedLeaderCheckInterval = isolatedLeaderCheckInterval; + this.isolatedLeaderCheckInterval = isolatedLeaderCheckInterval.toMillis(); + } + + public void setElectionTimeoutFactor(long electionTimeoutFactor){ + this.electionTimeoutFactor = electionTimeoutFactor; + electionTimeOutInterval = null; + } + + public void setCustomRaftPolicyImplementationClass(String customRaftPolicyImplementationClass){ + this.customRaftPolicyImplementationClass = customRaftPolicyImplementationClass; + } + + @Override + public String getCustomRaftPolicyImplementationClass() { + return customRaftPolicyImplementationClass; } @Override @@ -68,6 +113,12 @@ public class DefaultConfigParamsImpl implements ConfigParams { return snapshotBatchCount; } + @Override + public int getSnapshotDataThresholdPercentage() { + return snapshotDataThresholdPercentage; + } + + @Override public FiniteDuration getHeartBeatInterval() { return heartBeatInterval; @@ -75,8 +126,11 @@ public class DefaultConfigParamsImpl implements ConfigParams { @Override public FiniteDuration getElectionTimeOutInterval() { - // returns 2 times the heart beat interval - return getHeartBeatInterval().$times(2); + if(electionTimeOutInterval == null) { + electionTimeOutInterval = getHeartBeatInterval().$times(electionTimeoutFactor); + } + + return electionTimeOutInterval; } @Override @@ -86,7 +140,7 @@ public class DefaultConfigParamsImpl implements ConfigParams { @Override public int getSnapshotChunkSize() { - return SNAPSHOT_CHUNK_SIZE; + return snapshotChunkSize; } @Override @@ -95,7 +149,50 @@ public class DefaultConfigParamsImpl implements ConfigParams { } @Override - public FiniteDuration getIsolatedCheckInterval() { + public long getIsolatedCheckIntervalInMillis() { return isolatedLeaderCheckInterval; } + + @Override + public long getElectionTimeoutFactor() { + return electionTimeoutFactor; + } + + @Override + public RaftPolicy getRaftPolicy() { + return policySupplier.get(); + } + + private class PolicySupplier implements Supplier{ + @Override + public RaftPolicy get() { + if(Strings.isNullOrEmpty(DefaultConfigParamsImpl.this.customRaftPolicyImplementationClass)){ + LOG.debug("No custom RaftPolicy specified. Using DefaultRaftPolicy"); + return DefaultRaftPolicy.INSTANCE; + } + try { + String className = DefaultConfigParamsImpl.this.customRaftPolicyImplementationClass; + LOG.info("Trying to use custom RaftPolicy {}", className); + Class c = Class.forName(className); + RaftPolicy obj = (RaftPolicy)c.newInstance(); + return obj; + } catch (Exception e) { + if(LOG.isDebugEnabled()) { + LOG.error("Could not create custom raft policy, will stick with default", e); + } else { + LOG.error("Could not create custom raft policy, will stick with default : cause = {}", e.getMessage()); + } + } + return DefaultRaftPolicy.INSTANCE; + } + } + + @Override + public PeerAddressResolver getPeerAddressResolver() { + return peerAddressResolver; + } + + public void setPeerAddressResolver(@Nonnull PeerAddressResolver peerAddressResolver) { + this.peerAddressResolver = Preconditions.checkNotNull(peerAddressResolver); + } }