Skip to content

Commit

Permalink
Fix nits
Browse files Browse the repository at this point in the history
  • Loading branch information
jopemachine committed Feb 1, 2024
1 parent 003c296 commit 32ac6f4
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
1 change: 0 additions & 1 deletion examples/memstore/dynamic-members/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ async fn main() -> std::result::Result<(), Box<dyn std::error::Error>> {
.await
.unwrap();
let node_id = ticket.reserved_id;
println!("ticket.peers {:?}", ticket.peers);
cfg.initial_peers = Some(ticket.peers.clone().into());

let raft = Raft::bootstrap(
Expand Down
2 changes: 1 addition & 1 deletion raftify/src/raft_node/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -457,7 +457,7 @@ impl<
.clone()
.inner
.into_iter()
.filter(|(_, peer)| peer.role == InitialRole::Voter)
.filter(|(_, peer)| peer.role == InitialRole::Voter || peer.role == InitialRole::Leader)
.map(|(key, _)| key)
.collect::<Vec<_>>();

Expand Down
9 changes: 5 additions & 4 deletions raftify/src/raft_node/role.rs
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,11 @@ impl FromStr for InitialRole {
type Err = ();

fn from_str(s: &str) -> Result<Self, Self::Err> {
match s {
"Leader" => Ok(InitialRole::Leader),
"Voter" => Ok(InitialRole::Voter),
"Learner" => Ok(InitialRole::Learner),
let s = s.to_lowercase();
match s.as_str() {
"leader" => Ok(InitialRole::Leader),
"voter" => Ok(InitialRole::Voter),
"learner" => Ok(InitialRole::Learner),
_ => Err(()),
}
}
Expand Down

0 comments on commit 32ac6f4

Please sign in to comment.