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%2FRequestVote.java;h=d5a581aab8cdbe1d78f29da9b31ccf52025d6912;hb=4ecb8ecaf04594b3312a44d801423f515ea445b3;hp=fc6d48906a73b63f34eaf053792b2c3a0cb1c3c0;hpb=1554d7b1c046ec16505f291c0fd7bfd162935f8f;p=controller.git diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/RequestVote.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/RequestVote.java index fc6d48906a..d5a581aab8 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/RequestVote.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/RequestVote.java @@ -8,6 +8,11 @@ package org.opendaylight.controller.cluster.raft.messages; +import java.io.Externalizable; +import java.io.IOException; +import java.io.ObjectInput; +import java.io.ObjectOutput; + /** * Invoked by candidates to gather votes (§5.2). */ @@ -44,10 +49,52 @@ public class RequestVote extends AbstractRaftRPC { @Override public String toString() { - StringBuilder builder = new StringBuilder(); - builder.append("RequestVote [term=").append(getTerm()).append(", candidateId=").append(candidateId) - .append(", lastLogIndex=").append(lastLogIndex).append(", lastLogTerm=").append(lastLogTerm) - .append("]"); - return builder.toString(); + return "RequestVote [term=" + getTerm() + + ", candidateId=" + candidateId + + ", lastLogIndex=" + lastLogIndex + + ", lastLogTerm=" + lastLogTerm + + "]"; + } + + private Object writeReplace() { + return new Proxy(this); + } + + private static class Proxy implements Externalizable { + private static final long serialVersionUID = 1L; + + private RequestVote requestVote; + + // 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(RequestVote requestVote) { + this.requestVote = requestVote; + } + + @Override + public void writeExternal(ObjectOutput out) throws IOException { + out.writeLong(requestVote.getTerm()); + out.writeObject(requestVote.candidateId); + out.writeLong(requestVote.lastLogIndex); + out.writeLong(requestVote.lastLogTerm); + } + + @Override + public void readExternal(ObjectInput in) throws IOException, ClassNotFoundException { + long term = in.readLong(); + String candidateId = (String) in.readObject(); + long lastLogIndex = in.readLong(); + long lastLogTerm = in.readLong(); + + requestVote = new RequestVote(term, candidateId, lastLogIndex, lastLogTerm); + } + + private Object readResolve() { + return requestVote; + } } }