kamille
kamille
@alamb 😄 Very glad that it can help, and thanks for help! I am fixing comments and adding the unit tests, and will help more things about merging soon.
> I ran the benchmarks on my test machine and I see similar results > > Overall things get significantly better but a few get slower (e.g. `Q37`) but I...
Have refactored the codes to eliminate the unnecessary operations which may be the reason why some queries slower following the suggestion https://github.com/apache/datafusion/pull/11943#discussion_r1723425840 I did a small benchmark about q9 which...
Current progress: #### 1. Tried best to merge the accumulator loigc in two modes(flat and blocked) Mentioned in: https://github.com/apache/datafusion/pull/11943#discussion_r1727161296 But for using static dispatch to improve the performance, still keep...
Hi @alamb main comments https://github.com/apache/datafusion/pull/11943#pullrequestreview-2254643945 for this pr have been fixed, minding have a quick look? It would be appreciated. The detail about main progress: - Added a phase for...
> I am also curious what is the compatible matrix for all aggregation optimizations (like can skip-partial-aggregation and external-aggregation triggered in the same execution, for all combinations) Specifying them in...
As I think, can we run the basic aggregation without any optimizations enabled and use its output as expected first, and then we modify the options to enable different optimizations...
> > As I think, can we run the basic aggregation without any optimizations enabled and use its output as expected first, > > and then we modify the options...
> What would you suggest the next steps here be @Rachelint ? I personally plan to implement some necessary features of the framework firstly, like: - support test spilling -...