Skip to content

Commit d67b94b

Browse files
committed
Hide cast module and update internal access
Make the cast module private again by changing its visibility to module only. Provide crate-internal access to the helper function from expressions::mod, and update the planner to use this internal entry point instead of direct module exposure.
1 parent 05fe6a5 commit d67b94b

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

datafusion/physical-expr/src/expressions/mod.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
#[macro_use]
2121
mod binary;
2222
mod case;
23-
pub(crate) mod cast;
23+
mod cast;
2424
mod cast_column;
2525
mod column;
2626
mod dynamic_filters;
@@ -41,6 +41,7 @@ pub use crate::aggregate::stats::StatsType;
4141

4242
pub use binary::{BinaryExpr, binary, similar_to};
4343
pub use case::{CaseExpr, case};
44+
pub(crate) use cast::cast_with_target_field_and_options;
4445
pub use cast::{CastExpr, cast};
4546
pub use cast_column::CastColumnExpr;
4647
pub use column::{Column, col, with_new_schema};

datafusion/physical-expr/src/planner.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ pub fn create_physical_expr(
289289
Ok(expressions::case(expr, when_then_expr, else_expr)?)
290290
}
291291
Expr::Cast(Cast { expr, field }) => {
292-
expressions::cast::cast_with_target_field_and_options(
292+
expressions::cast_with_target_field_and_options(
293293
create_physical_expr(expr, input_dfschema, execution_props)?,
294294
input_schema,
295295
Arc::clone(field),

0 commit comments

Comments
 (0)