osipovartem opened a new pull request, #25373: URL: https://github.com/apache/datafusion/pull/25373
## Which issue does this PR close? Closes no issue. ## Rationale for this change `SqlToRel::parse_json_access` represents SQL colon access as a colon binary operator. For an Arrow Struct, that expression currently retains the complete Struct type and reaches an unsupported physical binary operator instead of accessing the requested field. ## What changes are included in this PR? `NestedFunctionPlanner` now lowers a static colon access on a Struct to the existing vectorized `get_field` UDF. Nested paths become nested `get_field` calls, which the existing simplifier can flatten and push toward scan leaves. Non-Struct colon expressions remain available to other planners. The tests cover nested Struct access and the non-Struct fallback. ## Are there any user-facing changes? SQL JSON-style access on typed Struct expressions can now be planned and executed through `get_field`. ## How was this change tested? - `cargo test -p datafusion-functions-nested planner::tests --lib` (2 passed) - `cargo fmt --all -- --check` A package clippy run reached an unrelated pre-existing `from_iter_instead_of_collect` warning in `datafusion/common/src/scalar/mod.rs`; the changed crate passed this same focused clippy command on the DataFusion 55 fork. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
