Code Review
/
controller.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
review
|
tree
raw
(from parent 1:
f501e16
)
Merge "InstallSnapshot does not pass the term to the protobuff instance"
author
Moiz Raja
<moraja@cisco.com>
Fri, 20 Feb 2015 00:13:32 +0000
(
00:13
+0000)
committer
Gerrit Code Review
<gerrit@opendaylight.org>
Fri, 20 Feb 2015 00:13:33 +0000
(
00:13
+0000)
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/InstallSnapshot.java
patch
|
blob
|
history
opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/LeaderTest.java
patch
|
blob
|
history
diff --git
a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/InstallSnapshot.java
b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/InstallSnapshot.java
index 119b43ce83acad53740566e50187411308d2fc69..13636f36d7594322432377b5156fece4d6279bd6 100644
(file)
--- a/
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/InstallSnapshot.java
+++ b/
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/messages/InstallSnapshot.java
@@
-73,6
+73,7
@@
public class InstallSnapshot extends AbstractRaftRPC {
public <T extends Object> Object toSerializable(){
InstallSnapshotMessages.InstallSnapshot.Builder builder = InstallSnapshotMessages.InstallSnapshot.newBuilder()
+ .setTerm(this.getTerm())
.setLeaderId(this.getLeaderId())
.setChunkIndex(this.getChunkIndex())
.setData(this.getData())
diff --git
a/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/LeaderTest.java
b/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/LeaderTest.java
index 7c37cbc502eb00db6b2067a2cf6cc6addf1f4f35..02c391f146630277c9fc60233193bb8108cd6961 100644
(file)
--- a/
opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/LeaderTest.java
+++ b/
opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/LeaderTest.java
@@
-424,8
+424,7
@@
public class LeaderTest extends AbstractRaftActorBehaviorTest {
assertEquals(snapshotIndex, installSnapshot.getLastIncludedIndex());
assertEquals(snapshotTerm, installSnapshot.getLastIncludedTerm());
- // FIXME - we don't set the term in the serialized message.
- //assertEquals(currentTerm, installSnapshot.getTerm());
+ assertEquals(currentTerm, installSnapshot.getTerm());
}
@Test