3

danolivo · danolivo · commit 00caa63110e3 · 2024-05-07T16:50:31.000+07:00
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c
@@ -1421,7 +1421,10 @@ grouping_planner(PlannerInfo *root, double tuple_fraction,
 		}
 		else if (parse->groupClause)
 		{
-			/* Preprocess regular GROUP BY clause, if any */
+			/*
+			 * Make a copy of origin groupClause because we are going to
+			 * remove redundant clauses.
+			 */
 			root->processed_groupClause = list_copy(parse->groupClause);
 			/* Remove any redundant GROUP BY columns */
 			remove_useless_groupby_columns(root);

Original file line number	Diff line number	Diff line change
`@@ -1421,7 +1421,10 @@ grouping_planner(PlannerInfo *root, double tuple_fraction,`
`1421`	`1421`	`}`
`1422`	`1422`	`else if (parse->groupClause)`
`1423`	`1423`	`{`
`1424`		`- /* Preprocess regular GROUP BY clause, if any */`
	`1424`	`+ /*`
	`1425`	`+ * Make a copy of origin groupClause because we are going to`
	`1426`	`+ * remove redundant clauses.`
	`1427`	`+ */`
`1425`	`1428`	`root->processed_groupClause = list_copy(parse->groupClause);`
`1426`	`1429`	`/* Remove any redundant GROUP BY columns */`
`1427`	`1430`	`remove_useless_groupby_columns(root);`