| From: | David Christensen <david(at)pgguru(dot)net> |
|---|---|
| To: | jian he <jian(dot)universality(at)gmail(dot)com> |
| Cc: | Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, Peter Eisentraut <peter(at)eisentraut(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Jelte Fennema-Nio <postgres(at)jeltef(dot)nl> |
| Subject: | Re: [PATCH] GROUP BY ALL |
| Date: | 2025-09-26 16:56:49 |
| Message-ID: | CAHM0NXg=yGzhn=H0_HiZ9u=Xq-40g1f+Bb=2e2LDxgd6a11bWw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Sep 26, 2025 at 11:38 AM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
>
> On Fri, Sep 26, 2025 at 11:46 PM David Christensen <david(at)pgguru(dot)net> wrote:
> >
> > >
> > > 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.)
> >
>
> hi.
> I only briefly browse the patch text file, so forgive me.
> seems missing deparse regress tests
>
> i think you may need one test like:
>
> create view v1 as SELECT b, COUNT(*) FROM t1 GROUP BY ALL;
> \sv v1
Thanks, good suggestion; not sure the appropriate final location for
this, but it did show a bug in the current patch.
David
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bertrand Drouvot | 2025-09-26 16:58:09 | Re: Report bytes and transactions actually sent downtream |
| Previous Message | jian he | 2025-09-26 16:37:29 | Re: [PATCH] GROUP BY ALL |