Skip to content

Commit 0a09e1c

Browse files
authored
feat: certralize schema parsing with literal coercion (delta-io#4106)
# Description We recently landed a fix to simplify predicates before using them... This PR consolidates that logic and adds another fix we requires in follow ups to properly coerce literals. This is since we are also translating the predicate to kernel where no coercion happens. Signed-off-by: Robert Pack <robstar.pack@gmail.com>
1 parent 5365923 commit 0a09e1c

7 files changed

Lines changed: 428 additions & 87 deletions

File tree

crates/core/src/delta_datafusion/table_provider.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1071,7 +1071,7 @@ pub(crate) fn simplify_expr(
10711071
df_schema: DFSchemaRef,
10721072
expr: Expr,
10731073
) -> Result<Arc<dyn PhysicalExpr>> {
1074-
let context = SimplifyContext::new(&session.execution_props()).with_schema(df_schema.clone());
1074+
let context = SimplifyContext::new(session.execution_props()).with_schema(df_schema.clone());
10751075
let simplifier = ExprSimplifier::new(context).with_max_cycles(10);
10761076
session.create_physical_expr(simplifier.simplify(expr)?, df_schema.as_ref())
10771077
}

0 commit comments

Comments
 (0)