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

Reply via email to