X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2Fmessages%2FAppendEntriesReply.java;h=e2f0ba9014c9e469b21599359db3007e7cb4e2be;hb=000960f6451af770f5463e41e1fb6defb6f3ab27;hp=7524d8f2326f0d490c09daa9842130e4b5218984;hpb=02bdbc1c781abc0b0b1d12dbfc1a19c316bebb98;p=controller.git diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/AppendEntriesReply.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/AppendEntriesReply.java index 7524d8f232..e2f0ba9014 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/AppendEntriesReply.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/AppendEntriesReply.java @@ -8,10 +8,17 @@ package org.opendaylight.controller.cluster.raft.messages; +import java.io.Externalizable; +import java.io.IOException; +import java.io.ObjectInput; +import java.io.ObjectOutput; +import org.opendaylight.controller.cluster.raft.RaftVersions; + /** - * Reply for the AppendEntriesRpc message + * Reply for the AppendEntries message. */ -public class AppendEntriesReply extends AbstractRaftRPC{ +public class AppendEntriesReply extends AbstractRaftRPC { + private static final long serialVersionUID = -7487547356392536683L; // true if follower contained entry matching // prevLogIndex and prevLogTerm @@ -28,17 +35,35 @@ public class AppendEntriesReply extends AbstractRaftRPC{ // responding private final String followerId; - public AppendEntriesReply(String followerId, long term, boolean success, long logLastIndex, long logLastTerm) { + private final short payloadVersion; + + private final short raftVersion; + + private final boolean forceInstallSnapshot; + + public AppendEntriesReply(String followerId, long term, boolean success, long logLastIndex, long logLastTerm, + short payloadVersion) { + this(followerId, term, success, logLastIndex, logLastTerm, payloadVersion, false); + } + + public AppendEntriesReply(String followerId, long term, boolean success, long logLastIndex, long logLastTerm, + short payloadVersion, boolean forceInstallSnapshot) { + this(followerId, term, success, logLastIndex, logLastTerm, payloadVersion, forceInstallSnapshot, + RaftVersions.CURRENT_VERSION); + + } + + private AppendEntriesReply(String followerId, long term, boolean success, long logLastIndex, long logLastTerm, + short payloadVersion, boolean forceInstallSnapshot, short raftVersion) { super(term); this.followerId = followerId; this.success = success; this.logLastIndex = logLastIndex; this.logLastTerm = logLastTerm; - } - - public long getTerm() { - return term; + this.payloadVersion = payloadVersion; + this.forceInstallSnapshot = forceInstallSnapshot; + this.raftVersion = raftVersion; } public boolean isSuccess() { @@ -56,4 +81,74 @@ public class AppendEntriesReply extends AbstractRaftRPC{ public String getFollowerId() { return followerId; } + + public short getPayloadVersion() { + return payloadVersion; + } + + public short getRaftVersion() { + return raftVersion; + } + + public boolean isForceInstallSnapshot() { + return forceInstallSnapshot; + } + + @Override + public String toString() { + return "AppendEntriesReply [term=" + getTerm() + ", success=" + success + ", followerId=" + followerId + + ", logLastIndex=" + logLastIndex + ", logLastTerm=" + logLastTerm + ", forceInstallSnapshot=" + + forceInstallSnapshot + ", payloadVersion=" + payloadVersion + ", raftVersion=" + raftVersion + "]"; + } + + private Object writeReplace() { + return new Proxy(this); + } + + private static class Proxy implements Externalizable { + private static final long serialVersionUID = 1L; + + private AppendEntriesReply appendEntriesReply; + + // checkstyle flags the public modifier as redundant which really doesn't make sense since it clearly isn't + // redundant. It is explicitly needed for Java serialization to be able to create instances via reflection. + @SuppressWarnings("checkstyle:RedundantModifier") + public Proxy() { + } + + Proxy(AppendEntriesReply appendEntriesReply) { + this.appendEntriesReply = appendEntriesReply; + } + + @Override + public void writeExternal(ObjectOutput out) throws IOException { + out.writeShort(appendEntriesReply.raftVersion); + out.writeLong(appendEntriesReply.getTerm()); + out.writeObject(appendEntriesReply.followerId); + out.writeBoolean(appendEntriesReply.success); + out.writeLong(appendEntriesReply.logLastIndex); + out.writeLong(appendEntriesReply.logLastTerm); + out.writeShort(appendEntriesReply.payloadVersion); + out.writeBoolean(appendEntriesReply.forceInstallSnapshot); + } + + @Override + public void readExternal(ObjectInput in) throws IOException, ClassNotFoundException { + short raftVersion = in.readShort(); + long term = in.readLong(); + String followerId = (String) in.readObject(); + boolean success = in.readBoolean(); + long logLastIndex = in.readLong(); + long logLastTerm = in.readLong(); + short payloadVersion = in.readShort(); + boolean forceInstallSnapshot = in.readBoolean(); + + appendEntriesReply = new AppendEntriesReply(followerId, term, success, logLastIndex, logLastTerm, + payloadVersion, forceInstallSnapshot, raftVersion); + } + + private Object readResolve() { + return appendEntriesReply; + } + } }