]> granicus.if.org Git - postgresql/commit
Fix create_scan_plan's handling of sortgrouprefs for physical tlists.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 11 Jul 2018 19:25:29 +0000 (15:25 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 11 Jul 2018 19:25:29 +0000 (15:25 -0400)
commitc3503202708c467f609163672ca09f2db082f3b5
tree40e462a44d4baf9ed609b5bcd787e800f96046f7
parent7c644b7d3f83b642aac16bc57dfe0de76a7045bb
Fix create_scan_plan's handling of sortgrouprefs for physical tlists.

We should only run apply_pathtarget_labeling_to_tlist if CP_LABEL_TLIST
was specified, because only in that case has use_physical_tlist checked
that the labeling will succeed; otherwise we may get an "ORDER/GROUP BY
expression not found in targetlist" error.  (This subsumes the previous
test about gating_clauses, because we reset "flags" to zero earlier
if there are gating clauses to apply.)

The only known case in which a failure can occur is with a ProjectSet
path directly atop a table scan path, although it seems likely that there
are other cases or will be such in future.  This means that the failure
is currently only visible in the v10 branch: 9.6 didn't have ProjectSet,
while in v11 and HEAD, apply_scanjoin_target_to_paths for some weird
reason is using create_projection_path not apply_projection_to_path,
masking the problem because there's a ProjectionPath in between.

Nonetheless this code is clearly wrong on its own terms, so back-patch
to 9.6 where this logic was introduced.

Per report from Regina Obe.

Discussion: https://postgr.es/m/001501d40f88$75186950$5f493bf0$@pcorp.us
src/backend/optimizer/plan/createplan.c
src/test/regress/expected/tsrf.out
src/test/regress/sql/tsrf.sql