alamb commented on issue #10181: URL: https://github.com/apache/datafusion/issues/10181#issuecomment-2070993030
> The "Expr walking" code now has to be aware of both the GetStructField and the ScalarUDF variants of field access. I think this can be controlled by the consumer -- for example if you are walking `Expr`s in lancedb, you can control when you transform `Expr::GetStructField` into `ScalarUDF` and depending on where you do your analysis you only have to check for one I think the idea is people might want to override `Expr::GetStructField`s semantics and they way they would do so is to rewrite it into a different function. I think this is especially compelling for supporting JSON/JSONB for example -- 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]
