Skip to content
GitLab
Menu
Projects
Groups
Snippets
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
Menu
Open sidebar
ruanhaishen
Willemt Raft
Commits
795c5e9e
Commit
795c5e9e
authored
Jan 24, 2016
by
Willem Thiart
Browse files
Fix: AE response term is set to new current term
parent
a2c2934b
Changes
2
Hide whitespace changes
Inline
Side-by-side
src/raft_server.c
View file @
795c5e9e
...
@@ -295,6 +295,7 @@ int raft_recv_appendentries(
...
@@ -295,6 +295,7 @@ int raft_recv_appendentries(
else
if
(
me
->
current_term
<
ae
->
term
)
else
if
(
me
->
current_term
<
ae
->
term
)
{
{
raft_set_current_term
(
me_
,
ae
->
term
);
raft_set_current_term
(
me_
,
ae
->
term
);
r
->
term
=
ae
->
term
;
raft_become_follower
(
me_
);
raft_become_follower
(
me_
);
}
}
else
if
(
ae
->
term
<
me
->
current_term
)
else
if
(
ae
->
term
<
me
->
current_term
)
...
...
tests/test_server.c
View file @
795c5e9e
...
@@ -693,6 +693,7 @@ void TestRaft_follower_recv_appendentries_updates_currentterm_if_term_gt_current
...
@@ -693,6 +693,7 @@ void TestRaft_follower_recv_appendentries_updates_currentterm_if_term_gt_current
/* appendentry has newer term, so we change our currentterm */
/* appendentry has newer term, so we change our currentterm */
raft_recv_appendentries
(
r
,
raft_get_node
(
r
,
2
),
&
ae
,
&
aer
);
raft_recv_appendentries
(
r
,
raft_get_node
(
r
,
2
),
&
ae
,
&
aer
);
CuAssertTrue
(
tc
,
1
==
aer
.
success
);
CuAssertTrue
(
tc
,
1
==
aer
.
success
);
CuAssertTrue
(
tc
,
2
==
aer
.
term
);
/* term has been updated */
/* term has been updated */
CuAssertTrue
(
tc
,
2
==
raft_get_current_term
(
r
));
CuAssertTrue
(
tc
,
2
==
raft_get_current_term
(
r
));
/* and leader has been updated */
/* and leader has been updated */
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment