xavlee commented on PR #24497: URL: https://github.com/apache/datafusion/pull/24497#issuecomment-5412439325
Hi Andrew, thanks for taking an initial look. Apologies this is still verymuch a draft. I agree that propagating another physical property through the plan would be a little complicated. I was hoping we would narrow the `group_contiguous_exprs` assertion s.t. it: - is declared explicitly by the data source - defaults to absent on every execution operator - may pass only through ProjectionExec when every expression maps - is consumed by the only the first AggregateExec (and not present on the aggregate output) -- 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]
