Skip to content

Commit 9c57164

Browse files
committed
minor
1 parent 8745cea commit 9c57164

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

crates/integrations/datafusion/src/table/mod.rs

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -194,11 +194,12 @@ impl TableProvider for IcebergTableProvider {
194194
};
195195

196196
// Step 2: Repartition for parallel processing
197-
let target_partitions = NonZeroUsize::new(state.config().target_partitions()).ok_or_else(|| {
198-
DataFusionError::Configuration(
199-
"target_partitions must be greater than 0".to_string(),
200-
)
201-
})?;
197+
let target_partitions =
198+
NonZeroUsize::new(state.config().target_partitions()).ok_or_else(|| {
199+
DataFusionError::Configuration(
200+
"target_partitions must be greater than 0".to_string(),
201+
)
202+
})?;
202203

203204
let repartitioned_plan = repartition(
204205
plan_with_partition,

crates/integrations/datafusion/src/task_writer.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,6 @@ use iceberg::writer::partitioning::unpartitioned_writer::UnpartitionedWriter;
6363
/// // Close and get data files
6464
/// let data_files = task_writer.close().await?;
6565
/// ```
66-
#[allow(dead_code)]
6766
pub(crate) struct TaskWriter<B: IcebergWriterBuilder> {
6867
/// The underlying writer (UnpartitionedWriter, FanoutWriter, or ClusteredWriter)
6968
writer: SupportedWriter<B>,
@@ -79,7 +78,6 @@ pub(crate) struct TaskWriter<B: IcebergWriterBuilder> {
7978
///
8079
/// This enum allows TaskWriter to work with different partitioning strategies
8180
/// while maintaining a unified interface.
82-
#[allow(dead_code)]
8381
enum SupportedWriter<B: IcebergWriterBuilder> {
8482
/// Writer for unpartitioned tables
8583
Unpartitioned(UnpartitionedWriter<B>),
@@ -89,7 +87,6 @@ enum SupportedWriter<B: IcebergWriterBuilder> {
8987
Clustered(ClusteredWriter<B>),
9088
}
9189

92-
#[allow(dead_code)]
9390
impl<B: IcebergWriterBuilder> TaskWriter<B> {
9491
/// Create a new TaskWriter.
9592
///

0 commit comments

Comments
 (0)