Remove fixme and a copy of group clause
A parse object has been copyed to subroot, and We do not free
root->simple_rel_array and root->simple_rte_array. So We do not
need to copy group clause.
@@ -4579,13 +4579,8 @@ add_second_stage_agg(PlannerInfo *root,
/*
* Since the rtable has changed, we had better recreate a RelOptInfo entry
- * for it. Make a copy of the groupClause since freeing the arrays can
- * pull out references still in use from underneath it.
- * We do not free root->simple_rel_array and root->simple_rte_array since
- * they are used by subroot.
- * GPDB_92_MERGE_FIXME: Do we still need to copy groupClause?
+ * for it.
*/
- root->parse->groupClause = copyObject(root->parse->groupClause);
rebuild_simple_rel_and_rte(root);
/*
