jayzhan211 commented on issue #9285: URL: https://github.com/apache/arrow-datafusion/issues/9285#issuecomment-1976668639
While I worked on ArrayElement, I found that `GetFieldAccessExpr::ListIndex` uses `array_element` in physical-expr https://github.com/apache/arrow-datafusion/blob/581fd98270e69221689ddd4c37566ef620a67167/datafusion/physical-expr/src/expressions/get_indexed_field.rs#L252-L257 ListRange likewise, which is kernel function in `function-arrays` crate, instead of converting ArrayRef to Expr, I'm thinking of making array function kernel public. @alamb Do you think we can make those array functions public? And, is importing `function-arrays` to crate `physical-expr` a concern like crate `optimize` is? If neither is a good idea, we may need to rewrite ListIndex and ListRange to array function early. -- 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]
