aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTrinity Pointard <trinity.pointard@gmail.com>2021-04-09 02:32:42 +0200
committerAlex Auvolat <alex@adnab.me>2021-05-03 22:11:41 +0200
commitf05bb111c2e7dd77f2b34b82892bbfa8e6a063c1 (patch)
tree0e093982ad6e30a440dcdbd9d8e47c4e75b9d198
parent88925ebe2210a8382b4d353fcab15d2b5c345efb (diff)
downloadgarage-f05bb111c2e7dd77f2b34b82892bbfa8e6a063c1.tar.gz
garage-f05bb111c2e7dd77f2b34b82892bbfa8e6a063c1.zip
fix clippy warnings on util and rpc
-rw-r--r--src/rpc/lib.rs1
-rw-r--r--src/rpc/membership.rs67
-rw-r--r--src/rpc/ring.rs7
-rw-r--r--src/rpc/rpc_client.rs2
-rw-r--r--src/rpc/rpc_server.rs6
-rw-r--r--src/util/data.rs2
-rw-r--r--src/util/error.rs4
-rw-r--r--src/util/lib.rs1
-rw-r--r--src/util/persister.rs6
9 files changed, 45 insertions, 51 deletions
diff --git a/src/rpc/lib.rs b/src/rpc/lib.rs
index 96561d0e..e787833c 100644
--- a/src/rpc/lib.rs
+++ b/src/rpc/lib.rs
@@ -1,3 +1,4 @@
+#![allow(clippy::upper_case_acronyms)]
//! Crate containing rpc related functions and types used in Garage
#[macro_use]
diff --git a/src/rpc/membership.rs b/src/rpc/membership.rs
index 5f7bbc96..ce4029f1 100644
--- a/src/rpc/membership.rs
+++ b/src/rpc/membership.rs
@@ -3,7 +3,7 @@ use std::collections::HashMap;
use std::fmt::Write as FmtWrite;
use std::io::{Read, Write};
use std::net::{IpAddr, SocketAddr};
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
use std::sync::atomic::{AtomicUsize, Ordering};
use std::sync::Arc;
use std::time::Duration;
@@ -198,15 +198,15 @@ impl Status {
}
}
-fn gen_node_id(metadata_dir: &PathBuf) -> Result<UUID, Error> {
- let mut id_file = metadata_dir.clone();
+fn gen_node_id(metadata_dir: &Path) -> Result<UUID, Error> {
+ let mut id_file = metadata_dir.to_path_buf();
id_file.push("node_id");
if id_file.as_path().exists() {
let mut f = std::fs::File::open(id_file.as_path())?;
let mut d = vec![];
f.read_to_end(&mut d)?;
if d.len() != 32 {
- return Err(Error::Message(format!("Corrupt node_id file")));
+ return Err(Error::Message("Corrupt node_id file".to_string()));
}
let mut id = [0u8; 32];
@@ -256,7 +256,7 @@ impl System {
let state_info = StateInfo {
hostname: gethostname::gethostname()
.into_string()
- .unwrap_or("<invalid utf-8>".to_string()),
+ .unwrap_or_else(|_| "<invalid utf-8>".to_string()),
};
let ring = Ring::new(net_config);
@@ -296,12 +296,12 @@ impl System {
match msg {
Message::Ping(ping) => self2.handle_ping(&addr, &ping).await,
- Message::PullStatus => self2.handle_pull_status(),
- Message::PullConfig => self2.handle_pull_config(),
+ Message::PullStatus => Ok(self2.handle_pull_status()),
+ Message::PullConfig => Ok(self2.handle_pull_config()),
Message::AdvertiseNodesUp(adv) => self2.handle_advertise_nodes_up(&adv).await,
Message::AdvertiseConfig(adv) => self2.handle_advertise_config(&adv).await,
- _ => Err(Error::BadRPC(format!("Unexpected RPC message"))),
+ _ => Err(Error::BadRPC("Unexpected RPC message".to_string())),
}
}
});
@@ -358,13 +358,13 @@ impl System {
) {
let self2 = self.clone();
self.background
- .spawn_worker(format!("discovery loop"), |stop_signal| {
+ .spawn_worker("discovery loop".to_string(), |stop_signal| {
self2.discovery_loop(peers, consul_host, consul_service_name, stop_signal)
});
let self2 = self.clone();
self.background
- .spawn_worker(format!("ping loop"), |stop_signal| {
+ .spawn_worker("ping loop".to_string(), |stop_signal| {
self2.ping_loop(stop_signal)
});
}
@@ -424,7 +424,6 @@ impl System {
warn!("Node {:?} seems to be down.", id);
if !ring.config.members.contains_key(id) {
info!("Removing node {:?} from status (not in config and not responding to pings anymore)", id);
- drop(st);
status.nodes.remove(&id);
has_changes = true;
}
@@ -438,7 +437,7 @@ impl System {
self.update_status(&update_locked, status).await;
drop(update_locked);
- if to_advertise.len() > 0 {
+ if !to_advertise.is_empty() {
self.broadcast(Message::AdvertiseNodesUp(to_advertise), PING_TIMEOUT)
.await;
}
@@ -474,15 +473,13 @@ impl System {
Ok(self.make_ping())
}
- fn handle_pull_status(&self) -> Result<Message, Error> {
- Ok(Message::AdvertiseNodesUp(
- self.status.borrow().to_serializable_membership(self),
- ))
+ fn handle_pull_status(&self) -> Message {
+ Message::AdvertiseNodesUp(self.status.borrow().to_serializable_membership(self))
}
- fn handle_pull_config(&self) -> Result<Message, Error> {
+ fn handle_pull_config(&self) -> Message {
let ring = self.ring.borrow().clone();
- Ok(Message::AdvertiseConfig(ring.config.clone()))
+ Message::AdvertiseConfig(ring.config.clone())
}
async fn handle_advertise_nodes_up(
@@ -530,7 +527,7 @@ impl System {
self.update_status(&update_lock, status).await;
drop(update_lock);
- if to_ping.len() > 0 {
+ if !to_ping.is_empty() {
self.background
.spawn_cancellable(self.clone().ping_nodes(to_ping).map(Ok));
}
@@ -576,8 +573,8 @@ impl System {
self.clone().ping_nodes(ping_addrs).await;
select! {
- _ = restart_at.fuse() => (),
- _ = stop_signal.changed().fuse() => (),
+ _ = restart_at.fuse() => {},
+ _ = stop_signal.changed().fuse() => {},
}
}
}
@@ -595,7 +592,7 @@ impl System {
};
while !*stop_signal.borrow() {
- let not_configured = self.ring.borrow().config.members.len() == 0;
+ let not_configured = self.ring.borrow().config.members.is_empty();
let no_peers = self.status.borrow().nodes.len() < 3;
let bad_peers = self
.status
@@ -613,11 +610,8 @@ impl System {
.map(|ip| (*ip, None))
.collect::<Vec<_>>();
- match self.persist_status.load_async().await {
- Ok(peers) => {
- ping_list.extend(peers.iter().map(|x| (x.addr, Some(x.id))));
- }
- _ => (),
+ if let Ok(peers) = self.persist_status.load_async().await {
+ ping_list.extend(peers.iter().map(|x| (x.addr, Some(x.id))));
}
if let Some((consul_host, consul_service_name)) = &consul_config {
@@ -636,12 +630,14 @@ impl System {
let restart_at = tokio::time::sleep(DISCOVERY_INTERVAL);
select! {
- _ = restart_at.fuse() => (),
- _ = stop_signal.changed().fuse() => (),
+ _ = restart_at.fuse() => {},
+ _ = stop_signal.changed().fuse() => {},
}
}
}
+ // for some reason fixing this is causing compilation error, see https://github.com/rust-lang/rust-clippy/issues/7052
+ #[allow(clippy::manual_async_fn)]
fn pull_status(
self: Arc<Self>,
peer: UUID,
@@ -672,18 +668,15 @@ impl System {
let mut list = status.to_serializable_membership(&self);
// Combine with old peer list to make sure no peer is lost
- match self.persist_status.load_async().await {
- Ok(old_list) => {
- for pp in old_list {
- if !list.iter().any(|np| pp.id == np.id) {
- list.push(pp);
- }
+ if let Ok(old_list) = self.persist_status.load_async().await {
+ for pp in old_list {
+ if !list.iter().any(|np| pp.id == np.id) {
+ list.push(pp);
}
}
- _ => (),
}
- if list.len() > 0 {
+ if !list.is_empty() {
info!("Persisting new peer list ({} peers)", list.len());
self.persist_status
.save_async(&list)
diff --git a/src/rpc/ring.rs b/src/rpc/ring.rs
index bffd7f1f..d371bb64 100644
--- a/src/rpc/ring.rs
+++ b/src/rpc/ring.rs
@@ -141,8 +141,7 @@ impl Ring {
if i_round >= node_info.capacity {
continue;
}
- for pos2 in *pos..q.len() {
- let qv = q[pos2];
+ for (pos2, &qv) in q.iter().enumerate().skip(*pos) {
if partitions[qv].len() != rep {
continue;
}
@@ -205,7 +204,7 @@ impl Ring {
for (i, entry) in self.ring.iter().enumerate() {
ret.push((i as u16, entry.location));
}
- if ret.len() > 0 {
+ if !ret.is_empty() {
assert_eq!(ret[0].1, [0u8; 32].into());
}
@@ -234,6 +233,6 @@ impl Ring {
assert_eq!(partition_top & PARTITION_MASK_U16, top & PARTITION_MASK_U16);
assert!(n <= partition.nodes.len());
- partition.nodes[..n].iter().cloned().collect::<Vec<_>>()
+ partition.nodes[..n].to_vec()
}
}
diff --git a/src/rpc/rpc_client.rs b/src/rpc/rpc_client.rs
index 8a6cc721..f68e4c03 100644
--- a/src/rpc/rpc_client.rs
+++ b/src/rpc/rpc_client.rs
@@ -240,7 +240,7 @@ impl<M: RpcMessage> RpcAddrClient<M> {
pub fn new(http_client: Arc<RpcHttpClient>, path: String) -> Self {
Self {
phantom: PhantomData::default(),
- http_client: http_client,
+ http_client,
path,
}
}
diff --git a/src/rpc/rpc_server.rs b/src/rpc/rpc_server.rs
index 4419a6f0..55d97170 100644
--- a/src/rpc/rpc_server.rs
+++ b/src/rpc/rpc_server.rs
@@ -57,7 +57,7 @@ where
trace!(
"Request message: {}",
serde_json::to_string(&msg)
- .unwrap_or("<json error>".into())
+ .unwrap_or_else(|_| "<json error>".into())
.chars()
.take(100)
.collect::<String>()
@@ -123,7 +123,7 @@ impl RpcServer {
req: Request<Body>,
addr: SocketAddr,
) -> Result<Response<Body>, Error> {
- if req.method() != &Method::POST {
+ if req.method() != Method::POST {
let mut bad_request = Response::default();
*bad_request.status_mut() = StatusCode::BAD_REQUEST;
return Ok(bad_request);
@@ -201,7 +201,7 @@ impl RpcServer {
.get_ref()
.0
.peer_addr()
- .unwrap_or(([0, 0, 0, 0], 0).into());
+ .unwrap_or_else(|_| ([0, 0, 0, 0], 0).into());
let self_arc = self_arc.clone();
async move {
Ok::<_, Error>(service_fn(move |req: Request<Body>| {
diff --git a/src/util/data.rs b/src/util/data.rs
index 34ee8a18..56c7ab56 100644
--- a/src/util/data.rs
+++ b/src/util/data.rs
@@ -72,7 +72,7 @@ impl FixedBytes32 {
&mut self.0[..]
}
/// Copy to a slice
- pub fn to_vec(&self) -> Vec<u8> {
+ pub fn to_vec(self) -> Vec<u8> {
self.0.to_vec()
}
/// Try building a FixedBytes32 from a slice
diff --git a/src/util/error.rs b/src/util/error.rs
index 32dccbe6..2b862269 100644
--- a/src/util/error.rs
+++ b/src/util/error.rs
@@ -93,12 +93,12 @@ impl From<sled::transaction::TransactionError<Error>> for Error {
impl<T> From<tokio::sync::watch::error::SendError<T>> for Error {
fn from(_e: tokio::sync::watch::error::SendError<T>) -> Error {
- Error::Message(format!("Watch send error"))
+ Error::Message("Watch send error".to_string())
}
}
impl<T> From<tokio::sync::mpsc::error::SendError<T>> for Error {
fn from(_e: tokio::sync::mpsc::error::SendError<T>) -> Error {
- Error::Message(format!("MPSC send error"))
+ Error::Message("MPSC send error".to_string())
}
}
diff --git a/src/util/lib.rs b/src/util/lib.rs
index c080e3a3..15d020cc 100644
--- a/src/util/lib.rs
+++ b/src/util/lib.rs
@@ -1,3 +1,4 @@
+#![allow(clippy::upper_case_acronyms)]
//! Crate containing common functions and types used in Garage
#[macro_use]
diff --git a/src/util/persister.rs b/src/util/persister.rs
index 93b7cdf4..9e1a1910 100644
--- a/src/util/persister.rs
+++ b/src/util/persister.rs
@@ -1,5 +1,5 @@
use std::io::{Read, Write};
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
use tokio::io::{AsyncReadExt, AsyncWriteExt};
@@ -18,8 +18,8 @@ impl<T> Persister<T>
where
T: Serialize + for<'de> Deserialize<'de>,
{
- pub fn new(base_dir: &PathBuf, file_name: &str) -> Self {
- let mut path = base_dir.clone();
+ pub fn new(base_dir: &Path, file_name: &str) -> Self {
+ let mut path = base_dir.to_path_buf();
path.push(file_name);
Self {
path,