Rachelint commented on PR #14301: URL: https://github.com/apache/datafusion/pull/14301#issuecomment-2614356225
> @logan-keede I think it nice to incrementally split cases like that. Maybe we can start from the commonest ones(count, sum, avg, min/max and so on...), and the safe way may be one pr just involve one function of them? And in this pr, we just extract the common table creating sql, and rename the `aggregate.slt` to `aggregate_supplement.slt`? After refactoring, it may be like: ``` -- aggregate -- init_data.slt.part -- aggregate_supplement.slt ``` -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org