From: David Rowley <drowley@postgresql.org>
Date: Wed, 3 Apr 2024 02:39:29 +0000 (+1300)
Subject: Don't adjust ressortgroupref in generate_setop_child_grouplist()
X-Git-Tag: REL_17_BETA1~438
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7487044d6ccf183e53d2633024595779922dbee2;p=thirdparty%2Fpostgresql.git

Don't adjust ressortgroupref in generate_setop_child_grouplist()

This is already done inside assignSortGroupRef(), therefore is
redundant.

Oversight from 66c0185a3.

Reported-by: Tom Lane
Discussion: https://postgr.es/m/3703023.1711654574@sss.pgh.pa.us
---

diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c
index d4c13a518fd..5320da51a06 100644
--- a/src/backend/optimizer/plan/planner.c
+++ b/src/backend/optimizer/plan/planner.c
@@ -7959,7 +7959,6 @@ generate_setop_child_grouplist(SetOperationStmt *op, List *targetlist)
 
 		/* assign a tleSortGroupRef, or reuse the existing one */
 		sgc->tleSortGroupRef = assignSortGroupRef(tle, targetlist);
-		tle->ressortgroupref = sgc->tleSortGroupRef;
 	}
 	Assert(lg == NULL);
 	return grouplist;