From ef7649d9d82e7bf4ccb29814b244bebf168acb18 Mon Sep 17 00:00:00 2001 From: "R. Tyler Croy" Date: Sat, 29 Apr 2023 12:42:09 -0700 Subject: [PATCH] Update rust/src/operations/transaction/state.rs Co-authored-by: Will Jones --- rust/src/operations/transaction/state.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/rust/src/operations/transaction/state.rs b/rust/src/operations/transaction/state.rs index 1d69ee7ca3..c1bc4ffb90 100644 --- a/rust/src/operations/transaction/state.rs +++ b/rust/src/operations/transaction/state.rs @@ -43,8 +43,7 @@ impl DeltaTableState { .map(|f| { let field = ArrowField::try_from(f)?; let corrected = match field.data_type() { - // TODO Datafusion seems to currently not implement comparsion kernes for booleans in - // dict arrays, so we have to omit converting that type for now. + // Dictionary encoding columns bools is a waste of space DataType::Boolean => field.data_type().clone(), _ => wrap_partition_type_in_dict(field.data_type().clone()), };