Re: Partial aggregates pushdown - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Partial aggregates pushdown
Date
Msg-id 20221207185943.g3pkqseovvrk2n2e@awork3.anarazel.de
Whole thread Raw
In response to RE: Partial aggregates pushdown  ("Fujii.Yuki@df.MitsubishiElectric.co.jp" <Fujii.Yuki@df.MitsubishiElectric.co.jp>)
Responses RE: Partial aggregates pushdown  ("Fujii.Yuki@df.MitsubishiElectric.co.jp" <Fujii.Yuki@df.MitsubishiElectric.co.jp>)
List pgsql-hackers
Hi,

On 2022-12-05 02:03:49 +0000, Fujii.Yuki@df.MitsubishiElectric.co.jp wrote:
> > Attaching minor fixes. I haven't proof-read all comments (but perhaps, they
> > need attention from some native speaker).
> Thank you. I fixed according to your patch.
> And I fixed have proof-read all comments and messages.

cfbot complains about some compiler warnings when building with clang:
https://cirrus-ci.com/task/6606268580757504

deparse.c:3459:22: error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality]
        if ((node->aggsplit == AGGSPLIT_SIMPLE)) {
             ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~
deparse.c:3459:22: note: remove extraneous parentheses around the comparison to silence this warning
        if ((node->aggsplit == AGGSPLIT_SIMPLE)) {
            ~               ^                 ~
deparse.c:3459:22: note: use '=' to turn this equality comparison into an assignment
        if ((node->aggsplit == AGGSPLIT_SIMPLE)) {
                            ^~
                            =

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: daitch_mokotoff module
Next
From: Andres Freund
Date:
Subject: Re: Report checkpoint progress with pg_stat_progress_checkpoint (was: Report checkpoint progress in server logs)