Skip to content

Commit a821780

Browse files
committed
Update overlooked comment for Gather Merge.
Commit 355d399 probably should have done this, but nobody noticed that it was needed.
1 parent bce352f commit a821780

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/backend/optimizer/plan/createplan.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5390,9 +5390,9 @@ make_sort(Plan *lefttree, int numCols,
53905390
* prepare_sort_from_pathkeys
53915391
* Prepare to sort according to given pathkeys
53925392
*
5393-
* This is used to set up for both Sort and MergeAppend nodes. It calculates
5394-
* the executor's representation of the sort key information, and adjusts the
5395-
* plan targetlist if needed to add resjunk sort columns.
5393+
* This is used to set up for Sort, MergeAppend, and Gather Merge nodes. It
5394+
* calculates the executor's representation of the sort key information, and
5395+
* adjusts the plan targetlist if needed to add resjunk sort columns.
53965396
*
53975397
* Input parameters:
53985398
* 'lefttree' is the plan node which yields input tuples
@@ -5416,7 +5416,7 @@ make_sort(Plan *lefttree, int numCols,
54165416
*
54175417
* If the pathkeys include expressions that aren't simple Vars, we will
54185418
* usually need to add resjunk items to the input plan's targetlist to
5419-
* compute these expressions, since the Sort/MergeAppend node itself won't
5419+
* compute these expressions, since a Sort or MergeAppend node itself won't
54205420
* do any such calculations. If the input plan type isn't one that can do
54215421
* projections, this means adding a Result node just to do the projection.
54225422
* However, the caller can pass adjust_tlist_in_place = TRUE to force the

0 commit comments

Comments
 (0)