This is an automated email from the ASF dual-hosted git repository.
etseidl pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/main by this push:
new 8e669e7688 Fix linting issues missed by #8506 (#8581)
8e669e7688 is described below
commit 8e669e7688ab7dfaeb90aba6e4a41f4a25e1f9ef
Author: Ed Seidl <[email protected]>
AuthorDate: Thu Oct 9 18:12:20 2025 -0700
Fix linting issues missed by #8506 (#8581)
---
parquet/src/arrow/arrow_writer/byte_array.rs | 2 +-
parquet/src/arrow/arrow_writer/mod.rs | 13 ++++++-------
parquet/src/column/writer/encoder.rs | 2 +-
parquet/src/geospatial/accumulator.rs | 10 ++++++----
parquet/tests/geospatial.rs | 19 +++++++++----------
5 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/parquet/src/arrow/arrow_writer/byte_array.rs
b/parquet/src/arrow/arrow_writer/byte_array.rs
index 4f3efadbf3..228d229b30 100644
--- a/parquet/src/arrow/arrow_writer/byte_array.rs
+++ b/parquet/src/arrow/arrow_writer/byte_array.rs
@@ -23,7 +23,7 @@ use crate::encodings::encoding::{DeltaBitPackEncoder,
Encoder};
use crate::encodings::rle::RleEncoder;
use crate::errors::{ParquetError, Result};
use crate::file::properties::{EnabledStatistics, WriterProperties,
WriterVersion};
-use crate::geospatial::accumulator::{try_new_geo_stats_accumulator,
GeoStatsAccumulator};
+use crate::geospatial::accumulator::{GeoStatsAccumulator,
try_new_geo_stats_accumulator};
use crate::geospatial::statistics::GeospatialStatistics;
use crate::schema::types::ColumnDescPtr;
use crate::util::bit_util::num_required_bits;
diff --git a/parquet/src/arrow/arrow_writer/mod.rs
b/parquet/src/arrow/arrow_writer/mod.rs
index 38ac190e99..020f99c276 100644
--- a/parquet/src/arrow/arrow_writer/mod.rs
+++ b/parquet/src/arrow/arrow_writer/mod.rs
@@ -4161,13 +4161,12 @@ mod tests {
true,
)]));
let parquet_schema = Type::group_type_builder("root")
- .with_fields(vec![Type::primitive_type_builder(
- "integers",
- crate::basic::Type::INT64,
- )
- .build()
- .unwrap()
- .into()])
+ .with_fields(vec![
+ Type::primitive_type_builder("integers",
crate::basic::Type::INT64)
+ .build()
+ .unwrap()
+ .into(),
+ ])
.build()
.unwrap();
let parquet_schema_descr =
SchemaDescriptor::new(parquet_schema.into());
diff --git a/parquet/src/column/writer/encoder.rs
b/parquet/src/column/writer/encoder.rs
index 419bd13177..1cf116dc3c 100644
--- a/parquet/src/column/writer/encoder.rs
+++ b/parquet/src/column/writer/encoder.rs
@@ -28,7 +28,7 @@ use crate::data_type::private::ParquetValueType;
use crate::encodings::encoding::{DictEncoder, Encoder, get_encoder};
use crate::errors::{ParquetError, Result};
use crate::file::properties::{EnabledStatistics, WriterProperties};
-use crate::geospatial::accumulator::{try_new_geo_stats_accumulator,
GeoStatsAccumulator};
+use crate::geospatial::accumulator::{GeoStatsAccumulator,
try_new_geo_stats_accumulator};
use crate::geospatial::statistics::GeospatialStatistics;
use crate::schema::types::{ColumnDescPtr, ColumnDescriptor};
diff --git a/parquet/src/geospatial/accumulator.rs
b/parquet/src/geospatial/accumulator.rs
index e86452fc8e..daf7aa959c 100644
--- a/parquet/src/geospatial/accumulator.rs
+++ b/parquet/src/geospatial/accumulator.rs
@@ -294,10 +294,12 @@ mod test {
// We should not be able to initialize a global accumulator after
we've initialized at least
// one accumulator
- assert!(init_geo_stats_accumulator_factory(Arc::new(
- DefaultGeoStatsAccumulatorFactory::default()
- ))
- .is_err())
+ assert!(
+ init_geo_stats_accumulator_factory(Arc::new(
+ DefaultGeoStatsAccumulatorFactory::default()
+ ))
+ .is_err()
+ )
}
#[cfg(feature = "geospatial")]
diff --git a/parquet/tests/geospatial.rs b/parquet/tests/geospatial.rs
index 53a947327e..614acbcf45 100644
--- a/parquet/tests/geospatial.rs
+++ b/parquet/tests/geospatial.rs
@@ -132,11 +132,11 @@ mod test {
use std::{iter::zip, sync::Arc};
- use arrow_array::{create_array, ArrayRef, BinaryArray, RecordBatch};
+ use arrow_array::{ArrayRef, BinaryArray, RecordBatch, create_array};
use arrow_schema::{DataType, Field, Schema};
use bytes::Bytes;
use parquet::{
- arrow::{arrow_writer::ArrowWriterOptions, ArrowWriter},
+ arrow::{ArrowWriter, arrow_writer::ArrowWriterOptions},
column::reader::ColumnReader,
data_type::{ByteArray, ByteArrayType},
file::{
@@ -415,14 +415,13 @@ mod test {
fn parquet_schema_geometry() -> Type {
Type::group_type_builder("root")
- .with_fields(vec![Type::primitive_type_builder(
- "geo",
- parquet::basic::Type::BYTE_ARRAY,
- )
- .with_logical_type(Some(LogicalType::Geometry { crs: None }))
- .build()
- .unwrap()
- .into()])
+ .with_fields(vec![
+ Type::primitive_type_builder("geo",
parquet::basic::Type::BYTE_ARRAY)
+ .with_logical_type(Some(LogicalType::Geometry { crs: None
}))
+ .build()
+ .unwrap()
+ .into(),
+ ])
.build()
.unwrap()
}
