From 6c9517295f51eb815670fb929a2cbb7be517b345 Mon Sep 17 00:00:00 2001
From: Tomas Vondra <tomas.vondra@postgresql.org>
Date: Sun, 13 Nov 2022 18:15:52 +0100
Subject: [PATCH v21 6/6] review comments

---
 src/backend/optimizer/path/allpaths.c         | 2 +-
 src/backend/utils/misc/postgresql.conf.sample | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c
index 1055ea70940..05192ca549a 100644
--- a/src/backend/optimizer/path/allpaths.c
+++ b/src/backend/optimizer/path/allpaths.c
@@ -3352,7 +3352,7 @@ generate_grouping_paths(PlannerInfo *root, RelOptInfo *rel_grouped,
 						RelOptInfo *rel_plain, RelAggInfo *agg_info)
 {
 	ListCell   *lc;
-	Path	   *path;
+	Path	   *path;	/* XXX why declare at this level, not in the loops */
 
 	if (IS_DUMMY_REL(rel_plain))
 	{
diff --git a/src/backend/utils/misc/postgresql.conf.sample b/src/backend/utils/misc/postgresql.conf.sample
index 310c5ff7744..89f944d83a1 100644
--- a/src/backend/utils/misc/postgresql.conf.sample
+++ b/src/backend/utils/misc/postgresql.conf.sample
@@ -389,7 +389,6 @@
 #enable_seqscan = on
 #enable_sort = on
 #enable_tidscan = on
-#enable_agg_pushdown = on
 
 # - Planner Cost Constants -
 
-- 
2.38.1

