abhinav-phi commented on issue #25106: URL: https://github.com/apache/datafusion/issues/25106#issuecomment-5711664600
thanks for the pointer joseph i reproduced this on current main and the aggregate case is indeed already covered the extract_leaf_expressions pass and the placement api added in 53 pull movetowardsleafnodes sub expressions out of aggregate args into a projection directly above the scan so today avg over get_field on a struct column already produces exactly the after plan from this issue octet_length on the other hand is still keepinplace by default so it stays inside the aggregate and the per function opt in is what 25025 is adding for the length family so withdrawing my claim here since the framework work is already done and what remains is tagging individual functions -- 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]
