diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-01-08 20:18:17 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-01-08 20:18:17 -0500 |
commit | 6c6c53d0be66fc9a0ae96a2a5e4be83710962be0 (patch) | |
tree | ffe8e0e089eb4c80cc52a7bff3fecbc58725b42c /src/backend/tcop/postgres.c | |
parent | 0402f24413c3e70f796562cb5459f33af071b0cc (diff) | |
download | postgresql-6c6c53d0be66fc9a0ae96a2a5e4be83710962be0.tar.gz postgresql-6c6c53d0be66fc9a0ae96a2a5e4be83710962be0.zip |
Fix "cannot accept a set" error when only some arms of a CASE return a set.
In commit c1352052ef1d4eeb2eb1d822a207ddc2d106cb13, I implemented an
optimization that assumed that a function's argument expressions would
either always return a set (ie multiple rows), or always not. This is
wrong however: we allow CASE expressions in which some arms return a set
of some type and others just return a scalar of that type. There may be
other examples as well. To fix, replace the run-time test of whether an
argument returned a set with a static precheck (expression_returns_set).
This adds a little bit of query startup overhead, but it seems barely
measurable.
Per bug #8228 from David Johnston. This has been broken since 8.0,
so patch all supported branches.
Diffstat (limited to 'src/backend/tcop/postgres.c')
0 files changed, 0 insertions, 0 deletions