Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Bug 5740: Remove Serializable where not necessary
[controller.git]
/
opendaylight
/
md-sal
/
sal-akka-raft
/
src
/
main
/
java
/
org
/
opendaylight
/
controller
/
cluster
/
raft
/
messages
/
ChangeServersVotingStatus.java
diff --git
a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/ChangeServersVotingStatus.java
b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/ChangeServersVotingStatus.java
index 0f4601297ce39e01caca86ad3bb3a71ae6035873..3dc36934f40b2e31b8f709890c894d38426096cc 100644
(file)
--- a/
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/ChangeServersVotingStatus.java
+++ b/
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/ChangeServersVotingStatus.java
@@
-8,7
+8,10
@@
package org.opendaylight.controller.cluster.raft.messages;
import com.google.common.base.Preconditions;
package org.opendaylight.controller.cluster.raft.messages;
import com.google.common.base.Preconditions;
+import com.google.common.collect.ImmutableSet;
import java.io.Serializable;
import java.io.Serializable;
+import java.util.Collection;
+import java.util.Collections;
import java.util.HashMap;
import java.util.Map;
import javax.annotation.Nonnull;
import java.util.HashMap;
import java.util.Map;
import javax.annotation.Nonnull;
@@
-22,17
+25,31
@@
public class ChangeServersVotingStatus implements Serializable {
private static final long serialVersionUID = 1L;
private final Map<String, Boolean> serverVotingStatusMap;
private static final long serialVersionUID = 1L;
private final Map<String, Boolean> serverVotingStatusMap;
+ private final Collection<String> serversVisited;
public ChangeServersVotingStatus(@Nonnull Map<String, Boolean> serverVotingStatusMap) {
public ChangeServersVotingStatus(@Nonnull Map<String, Boolean> serverVotingStatusMap) {
+ this(serverVotingStatusMap, Collections.emptySet());
+ }
+
+ public ChangeServersVotingStatus(@Nonnull Map<String, Boolean> serverVotingStatusMap,
+ @Nonnull Collection<String> serversVisited) {
this.serverVotingStatusMap = new HashMap<>(Preconditions.checkNotNull(serverVotingStatusMap));
this.serverVotingStatusMap = new HashMap<>(Preconditions.checkNotNull(serverVotingStatusMap));
+ this.serversVisited = ImmutableSet.copyOf(Preconditions.checkNotNull(serversVisited));
}
}
+ @Nonnull
public Map<String, Boolean> getServerVotingStatusMap() {
return serverVotingStatusMap;
}
public Map<String, Boolean> getServerVotingStatusMap() {
return serverVotingStatusMap;
}
+ @Nonnull
+ public Collection<String> getServersVisited() {
+ return serversVisited;
+ }
+
@Override
public String toString() {
@Override
public String toString() {
- return "ChangeServersVotingStatus [serverVotingStatusMap=" + serverVotingStatusMap + "]";
+ return "ChangeServersVotingStatus [serverVotingStatusMap=" + serverVotingStatusMap
+ + (serversVisited != null ? ", serversVisited=" + serversVisited : "") + "]";
}
}
}
}