postgresql/src/backend
Tom Lane 33343b862c Fix mishandling of FieldSelect-on-whole-row-Var in nested lateral queries.
If an inline-able SQL function taking a composite argument is used in a
LATERAL subselect, and the composite argument is a lateral reference,
the planner could fail with "variable not found in subplan target list",
as seen in bug #11703 from Karl Bartel.  (The outer function call used in
the bug report and in the committed regression test is not really necessary
to provoke the bug --- you can get it if you manually expand the outer
function into "LATERAL (SELECT inner_function(outer_relation))", too.)

The cause of this is that we generate the reltargetlist for the referenced
relation before doing eval_const_expressions() on the lateral sub-select's
expressions (cf find_lateral_references()), so what's scheduled to be
emitted by the referenced relation is a whole-row Var, not the simplified
single-column Var produced by optimizing the function's FieldSelect on the
whole-row Var.  Then setrefs.c fails to match up that lateral reference to
what's available from the outer scan.

Preserving the FieldSelect optimization in such cases would require either
major planner restructuring (to recursively do expression simplification
on sub-selects much earlier) or some amazingly ugly kluge to change the
reltargetlist of a possibly-already-planned relation.  It seems better
just to skip the optimization when the Var is from an upper query level;
the case is not so common that it's likely anyone will notice a few
wasted cycles.

AFAICT this problem only occurs for uplevel LATERAL references, so
back-patch to 9.3 where LATERAL was added.
2014-10-20 12:23:44 -04:00
..
access Message improvements 2014-10-12 01:02:56 -04:00
bootstrap Revert "Fix bogus %name-prefix option syntax in all our Bison files." 2014-05-28 19:21:01 -04:00
catalog pg_upgrade: prevent oid conflicts with new-cluster TOAST tables 2014-08-07 14:56:13 -04:00
commands Print planning time only in EXPLAIN ANALYZE, not plain EXPLAIN. 2014-10-15 18:50:16 -04:00
executor Assorted message fixes and improvements 2014-09-05 01:24:29 -04:00
foreign Update copyright for 2014 2014-01-07 16:05:30 -05:00
lib pgindent run for 9.4 2014-05-06 12:12:18 -04:00
libpq Message improvements 2014-10-12 01:02:56 -04:00
main Make fallback implementation of pg_memory_barrier() work. 2014-05-17 18:29:46 -04:00
nodes Avoid core dump in _outPathInfo() for Path without a parent RelOptInfo. 2014-10-17 22:33:04 -04:00
optimizer Fix mishandling of FieldSelect-on-whole-row-Var in nested lateral queries. 2014-10-20 12:23:44 -04:00
parser Support ALTER SYSTEM RESET command. 2014-09-12 23:15:04 +09:00
po Translation updates 2014-10-05 23:22:24 -04:00
port Fix spinlock implementation for some !solaris sparc platforms. 2014-09-09 23:36:32 +02:00
postmaster Eliminate one background-worker-related flag variable. 2014-10-04 22:15:06 -04:00
regex Fix incorrect search for "x?" style matches in creviterdissect(). 2014-09-23 20:26:21 -04:00
replication Improve documentation about binary/textual output mode for output plugins. 2014-10-01 13:21:59 +02:00
rewrite Assorted message improvements 2014-08-29 00:01:34 -04:00
snowball Fix ancient encoding error in hungarian.stop. 2014-06-10 22:48:31 -04:00
storage Don't let protected variable access to be reordered after spinlock release. 2014-10-14 10:05:17 +03:00
tcop Log ALTER SYSTEM statements as DDL 2014-09-22 20:52:56 -04:00
tsearch pgindent run for 9.4 2014-05-06 12:12:18 -04:00
utils Support timezone abbreviations that sometimes change. 2014-10-16 15:22:13 -04:00
.gitignore Add gitignore for mingw/cygwin build outputs 2011-06-09 18:11:47 +02:00
common.mk Remove maintainer-check target, fold into normal build 2013-10-10 20:11:56 -04:00
Makefile Get rid of use of dlltool in Mingw builds. 2014-02-11 12:56:20 -05:00
nls.mk Add libpgcommon to backend gettext source files 2013-10-19 13:49:05 -04:00