Skip to content

Commit e43e0cc

Browse files
committed
address clippy
1 parent d9603ff commit e43e0cc

File tree

4 files changed

+7
-14
lines changed

4 files changed

+7
-14
lines changed

crates/catalog/glue/src/utils.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ mod tests {
339339
&table_name,
340340
metadata_location,
341341
&metadata,
342-
&metadata.properties(),
342+
metadata.properties(),
343343
None,
344344
)?;
345345

crates/iceberg/src/spec/avro_util.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,7 @@ pub(crate) fn codec_from_str(codec: Option<&str>, level: Option<u8>) -> Codec {
8787
None => Codec::Null,
8888
Some(unknown) => {
8989
warn!(
90-
"Unrecognized compression codec '{}', using no compression (Codec::Null)",
91-
unknown
90+
"Unrecognized compression codec '{unknown}', using no compression (Codec::Null)"
9291
);
9392
Codec::Null
9493
}

crates/iceberg/src/spec/manifest/writer.rs

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -775,8 +775,7 @@ mod tests {
775775
let data_file = DataFileBuilder::default()
776776
.content(DataContentType::Data)
777777
.file_path(format!(
778-
"/very/long/path/to/data/directory/with/many/subdirectories/file_{}.parquet",
779-
i
778+
"/very/long/path/to/data/directory/with/many/subdirectories/file_{i}.parquet"
780779
))
781780
.file_format(DataFileFormat::Parquet)
782781
.partition(Struct::empty())
@@ -815,8 +814,7 @@ mod tests {
815814
let data_file = DataFileBuilder::default()
816815
.content(DataContentType::Data)
817816
.file_path(format!(
818-
"/very/long/path/to/data/directory/with/many/subdirectories/file_{}.parquet",
819-
i
817+
"/very/long/path/to/data/directory/with/many/subdirectories/file_{i}.parquet"
820818
))
821819
.file_format(DataFileFormat::Parquet)
822820
.partition(Struct::empty())
@@ -840,9 +838,7 @@ mod tests {
840838
// Verify compression is actually working
841839
assert!(
842840
compressed_size < uncompressed_size,
843-
"Compressed size ({}) should be less than uncompressed size ({})",
844-
compressed_size,
845-
uncompressed_size
841+
"Compressed size ({compressed_size}) should be less than uncompressed size ({uncompressed_size})"
846842
);
847843

848844
// Verify the compressed file can be read back correctly

crates/iceberg/src/spec/manifest_list.rs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2083,7 +2083,7 @@ mod test {
20832083
let mut entries = Vec::new();
20842084
for i in 0..100 {
20852085
entries.push(ManifestFile {
2086-
manifest_path: format!("/test/manifest{}.avro", i),
2086+
manifest_path: format!("/test/manifest{i}.avro"),
20872087
manifest_length: 1000 + i,
20882088
partition_spec_id: 0,
20892089
content: ManifestContentType::Data,
@@ -2153,9 +2153,7 @@ mod test {
21532153
// Verify compression is actually working
21542154
assert!(
21552155
compressed_size < uncompressed_size,
2156-
"Compressed size ({}) should be less than uncompressed size ({})",
2157-
compressed_size,
2158-
uncompressed_size
2156+
"Compressed size ({compressed_size}) should be less than uncompressed size ({uncompressed_size})"
21592157
);
21602158

21612159
// Verify the compressed file can be read back correctly

0 commit comments

Comments
 (0)