Re: WIP: Aggregation push-down

[legrand legrand]

Hello,

Thank you for this great feature !
I hope this will be reviewed/validated soon ;o)

Just a comment:
set enable_agg_pushdown to true;
isn't displayed in EXPLAIN (SETTINGS) syntax.


The following modification seems to fix that: 

src/backend/utils/misc/guc.c
 
                {"enable_agg_pushdown", PGC_USERSET, QUERY_TUNING_METHOD,
                        gettext_noop("Enables aggregation push-down."),
                        NULL,
                        GUC_EXPLAIN   <<<<--- line Added --->>>>
                },
                &enable_agg_pushdown,
                false,
                NULL, NULL, NULL
        },


then
postgres=# set enable_agg_pushdown = true;
SET
postgres=# explain (settings) select 1;
                QUERY PLAN
------------------------------------------
 Result  (cost=0.00..0.01 rows=1 width=4)
 Settings: enable_agg_pushdown = 'on'
(2 rows)


Regards
PAscal



--
Sent from: https://www.postgresql-archive.org/PostgreSQL-hackers-f1928748.html