- 09 Jan, 2018 2 commits
-
-
Willem Thiart authored
-
Willem Thiart authored
-
- 08 Jan, 2018 4 commits
-
-
Willem Thiart authored
-
Willem Thiart authored
-
Willem Thiart authored
-
Willem Thiart authored
-
- 29 Dec, 2017 2 commits
-
-
Willem Thiart authored
-
Willem Thiart authored
The raft_log API is now being fuzzed tested. This change fixes all of the bugs detected by the fuzzer so far.
-
- 21 Nov, 2017 13 commits
-
-
Willem Thiart authored
-
Li Wei authored
Move the declaration of raft_become_follower() from raft_private.h to raft.h, so that users may voluntarily give up leadership. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
User storage callbacks may return negative errors smaller than RAFT_ERR_LAST. These will be returned all the way to the API methods, which report the errors back to users, so that appropriate handlings may be performed. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
If an AE request's previous entry matches the follower's, then the follower doesn't need to truncate its log at prev_log_idx. When the follower checks if the other entries in the AE request are consistent with its local ones, it already takes care of truncating at any inconsistency. If any later entries not covered by this AE request are inconsistent, future AE requests shall truncate them. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
If raft_become_candidate() assigns a negative value e (i.e., -election_timeout < e < 0) to timeout_elapsed, and if this replica successfully becomes the leader, then this leader won't send heartbeats for request_timeout - e milliseconds. This may be longer than election_timeout and cause an unnecessary leadership change. This patch zeroes timeout_elapsed in raft_become_leader(), randomizes election timeouts in [election_timeout, 2 * election_timeout), and fixes raft_recv_appendentries() to avoid zeroing timeout_elapsed if the AE request has an older term. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
When a leader learns of a newer term and steps down, it should discard current_leader. Otherwise, a client request may be rejected with RAFT_ERR_NOT_LEADER and a hint pointing to this replica itself! For simplicity, this patche follows the Raft paper to discard current_leader whenever current_term changes. And, when receiving an AE with an up-to-date term, a server always updates current_leader, as this information is absolute, even if the previous entry doesn't match. This patch also fixes a test that has previously been failing due to incorrect node IDs. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
log_offer and log_poll should get log entry index rather than internal array index. That would match the callback comment in raft.h and would also match the log_pop case. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
Remind users' persist_term implementations to persist currentTerm and nil votedFor atomically. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
raft_recv_appendentries() doesn't need to check prev_log_idx against currentIndex, as this is already done implicitly by getting the entry at prev_log_idx. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
raft_recv_appendentries_response() should check r->term before checking r->current_idx. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
Check and handle memory allocation errors by reporting them back to users via NULL or RAFT_ERR_NOMEM return values. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Li Wei authored
log_append_entry() passes caller's raft_entry_t to log_offer. If the log_offer implementation sets the data buffer to different value, caller's raft_entry_t gets modified as a side effect. This is error-prone as callers are usually responsible for freeing the data buffer with such a log_offer implementation. This patch passes the raft_entry_t in the log to log_offer instead. Also, ety in raft_recv_entry() is no longer required. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
- 20 Nov, 2017 4 commits
-
-
Li Wei authored
When the calling replica isn't in the leader state, raft_recv_appendentries_response() should return RAFT_ERR_NOT_LEADER instead of -1. Signed-off-by:
Li Wei <wei.g.li@intel.com>
-
Willem Thiart authored
-
Anuj Kalia authored
-
Willem Thiart authored
-
- 22 Jun, 2017 1 commit
-
-
Willem Thiart authored
-
- 21 Jun, 2017 2 commits
-
-
Willem Thiart authored
Second if statement was redundant.
-
Willem Thiart authored
raft_send_requestvote and raft_send_appendentries now return proper error codes.
-
- 20 Jun, 2017 1 commit
-
-
Willem Thiart authored
Avoid deleting logs because of out-of-order/quick heartbeats.
-
- 30 May, 2017 1 commit
-
-
Willem Thiart authored
-
- 24 May, 2017 1 commit
-
-
Willem Thiart authored
Avoid for loop to increase performance when tallying votes for committing.
-
- 09 May, 2017 1 commit
-
-
Willem Thiart authored
Beforehand there were if statements to check if callbacks were provided by the user. This was only useful for making the test suites shorter. These if statements have been removed so that the core code is more concise.
-
- 25 Oct, 2016 2 commits
-
-
Willem Thiart authored
-
Maxim Kupriianov authored
Delete duplicated entires of the same two functions.
-
- 20 Oct, 2016 1 commit
-
-
javeme authored
remove the redundant assertion of 'next_idx' in raft_recv_appendentries_response().
-
- 20 Sep, 2016 1 commit
-
-
Willem Thiart authored
-
- 10 Sep, 2016 1 commit
-
-
Willem Thiart authored
-
- 31 Aug, 2016 1 commit
-
-
Willem Thiart authored
-
- 28 Aug, 2016 1 commit
-
-
Jordan Yelloz authored
This fixes the build in clang due to a bitwise inversion ambiguity.
-
- 11 Jun, 2016 1 commit
-
-
Willem Thiart authored
-