On Fri, Sep 26, 2025 at 6:16 AM David Christensen <[email protected]> wrote: > > > > > On Sep 26, 2025, at 3:45 AM, Andrey Borodin <[email protected]> wrote: > > > > > > > >> On 26 Sep 2025, at 12:02, Peter Eisentraut <[email protected]> wrote: > >> > >> maybe someone wants to take this up again. > > > > I've compared my old patch with David's, and his version looks better. > > If David is not going to work on this in nearest future - I'l like to pick > > up the work. > > Either way I'll join as reviewer. > > > > Thanks! > > I’m interested in picking it up again but would appreciate the review.
Here is a rebased version with a few more tests. I also changed the main check here to using `!contain_agg_clause` instead of `!IsA(Aggref))` directly. (This was defined in `optimizer/clauses.h`, but we already are pulling in `optimizer.h`, so it felt valid to me.) David
v2-0001-Add-GROUP-BY-ALL.patch
Description: Binary data
