Skip to content

Commit 1f01460

Browse files
authored
Address some clippy comments (#993)
1 parent 89a5a25 commit 1f01460

File tree

2 files changed

+8
-13
lines changed

2 files changed

+8
-13
lines changed

icechunk/src/format/manifest.rs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -378,11 +378,9 @@ fn lookup_ref<'a>(
378378
array_manifest: generated::ArrayManifest<'a>,
379379
coord: &ChunkIndices,
380380
) -> Option<generated::ChunkRef<'a>> {
381-
let res =
382-
array_manifest.refs().lookup_by_key(coord.0.as_slice(), |chunk_ref, coords| {
383-
chunk_ref.index().iter().cmp(coords.iter().copied())
384-
});
385-
res
381+
array_manifest.refs().lookup_by_key(coord.0.as_slice(), |chunk_ref, coords| {
382+
chunk_ref.index().iter().cmp(coords.iter().copied())
383+
})
386384
}
387385

388386
struct PayloadIterator {

icechunk/src/storage/object_store.rs

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -790,14 +790,11 @@ impl ObjectStoreBackend for S3ObjectStoreBackend {
790790
.with_access_key_id(credentials.access_key_id.clone())
791791
.with_secret_access_key(credentials.secret_access_key.clone());
792792

793-
let builder =
794-
if let Some(session_token) = credentials.session_token.as_ref() {
795-
builder.with_token(session_token.clone())
796-
} else {
797-
builder
798-
};
799-
800-
builder
793+
if let Some(session_token) = credentials.session_token.as_ref() {
794+
builder.with_token(session_token.clone())
795+
} else {
796+
builder
797+
}
801798
}
802799
Some(S3Credentials::Anonymous) => builder.with_skip_signature(true),
803800
// TODO: Support refreshable credentials

0 commit comments

Comments
 (0)