search

apache / datafusion

Apache DataFusion SQL Query Engine
https://datafusion.apache.org/
Apache License 2.0
5.29k stars 973 forks source link

Remove `Expr::GetIndexedField`, replace `Expr::{field,index,range}` with `FieldAccessor`, `IndexAccessor`, and `SliceAccessor` #10568

Closed jayzhan211 closed 1 week ago

jayzhan211 commented 2 weeks ago

Which issue does this PR close?

Closes #10374. Closes #10565

Rationale for this change

What changes are included in this PR?

After the change, you need to import 

use datafusion_functions::core::expr_ext::FieldAccessor;
use datafusion_functions_array::expr_ext::{IndexAccessor, SliceAccessor};

to use old expression api.

Are these changes tested?

Are there any user-facing changes?

jayzhan211 commented 1 week ago

Thanks, @alamb !