alamb commented on PR #8558: URL: https://github.com/apache/arrow-datafusion/pull/8558#issuecomment-1867613901
> We will think about this over the weekend and come back to you. Thank you > My preliminary thinking is Approach 3 could be a good stepping stone to Approach 2 if we feel like going direct to 2 would take too long I am also happy to help implement Approach 3 (as we already have some version of the code to do first_value / last_value downstream in IOx) I agree. In terms of getting to Approach 2, I think supporting shared CTEs (aka not actually re-computing a CTE each time it is referenced in the query) is required for approach 2 and would be a valuable feature in its own right for many DataFusion users so perhaps we could plan out that project -- 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]
