Skip to content

Commit d833449

Browse files
committed
Fix clippy issues
1 parent 4c0e16c commit d833449

File tree

3 files changed

+3
-6
lines changed

3 files changed

+3
-6
lines changed

parquet/src/column/writer/mod.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -3472,7 +3472,7 @@ mod tests {
34723472
let string_col = batch.column(0).as_string_opt::<i32>().unwrap();
34733473

34743474
let mut valid_count = 0;
3475-
for (_i, x) in string_col.iter().enumerate() {
3475+
for x in string_col.iter() {
34763476
if let Some(x) = x {
34773477
valid_count += 1;
34783478
assert_eq!(x, "parquet");

parquet/src/file/metadata/mod.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1225,7 +1225,7 @@ impl ColumnChunkMetaData {
12251225
let crypto_metadata = self
12261226
.column_crypto_metadata
12271227
.as_ref()
1228-
.map(|c| column_crypto_metadata::to_thrift(c));
1228+
.map(column_crypto_metadata::to_thrift);
12291229
#[cfg(not(feature = "encryption"))]
12301230
let crypto_metadata = None;
12311231

parquet/src/file/metadata/writer.rs

+1-4
Original file line numberDiff line numberDiff line change
@@ -277,10 +277,7 @@ impl<'a, W: Write> ThriftMetadataWriter<'a, W> {
277277
file_encryptor: &Arc<FileEncryptor>,
278278
) -> Result<crate::format::FileCryptoMetaData> {
279279
let properties = file_encryptor.properties();
280-
let supply_aad_prefix = match properties.aad_prefix() {
281-
Some(_) => Some(!properties.store_aad_prefix()),
282-
None => None,
283-
};
280+
let supply_aad_prefix = properties.aad_prefix().map(|_| !properties.store_aad_prefix());
284281
let encryption_algorithm = AesGcmV1 {
285282
aad_prefix: properties.aad_prefix().cloned(),
286283
aad_file_unique: Some(file_encryptor.aad_file_unique().clone()),

0 commit comments

Comments
 (0)