You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by vi...@apache.org on 2022/12/16 19:27:18 UTC
[arrow-rs] branch master updated: More clippy lint fixes (#3355)
This is an automated email from the ASF dual-hosted git repository.
viirya pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/master by this push:
new 89354ca7c More clippy lint fixes (#3355)
89354ca7c is described below
commit 89354ca7c3a66e05cefad77bd6350c8fad7b58bd
Author: Raphael Taylor-Davies <17...@users.noreply.github.com>
AuthorDate: Fri Dec 16 19:27:12 2022 +0000
More clippy lint fixes (#3355)
---
object_store/src/aws/client.rs | 2 +-
object_store/src/azure/client.rs | 2 +-
object_store/src/azure/credential.rs | 2 +-
object_store/src/local.rs | 4 ++--
4 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/object_store/src/aws/client.rs b/object_store/src/aws/client.rs
index ccc0a9c6b..d2d2aefa4 100644
--- a/object_store/src/aws/client.rs
+++ b/object_store/src/aws/client.rs
@@ -131,7 +131,7 @@ impl TryFrom<ListResponse> for ListResult {
let common_prefixes = value
.common_prefixes
.into_iter()
- .map(|x| Ok(Path::parse(&x.prefix)?))
+ .map(|x| Ok(Path::parse(x.prefix)?))
.collect::<Result<_>>()?;
let objects = value
diff --git a/object_store/src/azure/client.rs b/object_store/src/azure/client.rs
index b537f5edf..fedd85e3d 100644
--- a/object_store/src/azure/client.rs
+++ b/object_store/src/azure/client.rs
@@ -407,7 +407,7 @@ impl TryFrom<ListResultInternal> for ListResult {
.blob_prefix
.unwrap_or_default()
.into_iter()
- .map(|x| Ok(Path::parse(&x.name)?))
+ .map(|x| Ok(Path::parse(x.name)?))
.collect::<Result<_>>()?;
let objects = value
diff --git a/object_store/src/azure/credential.rs b/object_store/src/azure/credential.rs
index 721fcaea4..38e6e64f1 100644
--- a/object_store/src/azure/credential.rs
+++ b/object_store/src/azure/credential.rs
@@ -153,7 +153,7 @@ fn generate_authorization(
key: &str,
) -> String {
let str_to_sign = string_to_sign(h, u, method, account);
- let auth = hmac_sha256(base64::decode(key).unwrap(), &str_to_sign);
+ let auth = hmac_sha256(base64::decode(key).unwrap(), str_to_sign);
format!("SharedKey {}:{}", account, base64::encode(auth))
}
diff --git a/object_store/src/local.rs b/object_store/src/local.rs
index f7b7ad7dd..2ef87adbb 100644
--- a/object_store/src/local.rs
+++ b/object_store/src/local.rs
@@ -396,7 +396,7 @@ impl ObjectStore for LocalFileSystem {
None => self.config.root.to_file_path().unwrap(),
};
- let walkdir = WalkDir::new(&root_path)
+ let walkdir = WalkDir::new(root_path)
// Don't include the root directory itself
.min_depth(1)
.follow_links(true);
@@ -748,7 +748,7 @@ impl AsyncWrite for LocalUpload {
self.inner_state = LocalUploadState::Complete;
file.sync_all()?;
std::mem::drop(file);
- std::fs::rename(&staging_path, &self.dest)?;
+ std::fs::rename(staging_path, &self.dest)?;
Poll::Ready(Ok(()))
}
_ => {