aboutsummaryrefslogtreecommitdiff
path: root/src/backend/tcop/postgres.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2014-01-08 20:18:17 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2014-01-08 20:18:17 -0500
commit6c6c53d0be66fc9a0ae96a2a5e4be83710962be0 (patch)
treeffe8e0e089eb4c80cc52a7bff3fecbc58725b42c /src/backend/tcop/postgres.c
parent0402f24413c3e70f796562cb5459f33af071b0cc (diff)
downloadpostgresql-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