forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit8e617e2
committed
Fix whole-row Var evaluation to cope with resjunk columns (again).
When a whole-row Var is reading the result of a subquery, we need it toignore any "resjunk" columns that the subquery might have evaluated forGROUP BY or ORDER BY purposes. We've hacked this area before, in commit68e4099, but that fix only coveredwhole-row Vars of named composite types, not those of RECORD type; and itwas mighty klugy anyway, since it just assumed without checking that anyextra columns in the result must be resjunk. A proper fix requires gettinghold of the subquery's targetlist so we can actually see which columns areresjunk (whereupon we can use a JunkFilter to get rid of them). So bitethe bullet and add some infrastructure to make that possible.Per report from Andrew Dunstan and additional testing by Merlin Moncure.Back-patch to all supported branches. In 8.3, also back-patch commit292176a, which for some reason I hadnot done at the time, but it's a prerequisite for this change.1 parent3a0e4d3 commit8e617e2
File tree
6 files changed
+332
-178
lines changed- src
- backend/executor
- include/nodes
- test/regress
- expected
- sql
6 files changed
+332
-178
lines changed0 commit comments
Comments
(0)