3

danolivo · danolivo · commit fedb37dff611 · 2024-04-22T16:15:02.000+07:00
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c
@@ -1424,7 +1424,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
`@@ -1424,7 +1424,10 @@ grouping_planner(PlannerInfo *root, double tuple_fraction,`
`1424`	`1424`	`}`
`1425`	`1425`	`else if (parse->groupClause)`
`1426`	`1426`	`{`
`1427`		`- /* Preprocess regular GROUP BY clause, if any */`
	`1427`	`+ /*`
	`1428`	`+ * Make a copy of origin groupClause because we are going to`
	`1429`	`+ * remove redundant clauses.`
	`1430`	`+ */`
`1428`	`1431`	`root->processed_groupClause = list_copy(parse->groupClause);`
`1429`	`1432`	`/* Remove any redundant GROUP BY columns */`
`1430`	`1433`	`remove_useless_groupby_columns(root);`