aboutsummaryrefslogtreecommitdiff
path: root/src/backend/commands/variable.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2008-01-09 20:50:12 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2008-01-09 20:50:12 +0000
commit02b3f326b2fde426ece942c54b6b8976184f4c79 (patch)
tree579cf6b42e4eb3569f05429e29d5086d069a3e1e /src/backend/commands/variable.c
parent0a1e67bab37a7ad9099db5a7836ad64afbeb569d (diff)
downloadpostgresql-02b3f326b2fde426ece942c54b6b8976184f4c79.tar.gz
postgresql-02b3f326b2fde426ece942c54b6b8976184f4c79.zip
Fix a bug in 8.2.x that was exposed while investigating Kevin Grittner's
report of poor planning in 8.3: it's unsafe to push a constant across an outer join when the outer-join condition is delayed by lower outer joins, unless we recheck the outer-join condition at the upper outer join. 8.2.x doesn't really have the ability to tell whether this is the case or not, but fortunately it doesn't matter --- it seems most desirable to keep the join condition whether it's entirely redundant or not. However, it's usually mostly redundant, so force its selectivity to 1.0. It might be a good idea to back-patch this into 8.1 as well, but I'll refrain until/unless there's evidence that 8.1 actually fails on any cases that this would fix.
Diffstat (limited to 'src/backend/commands/variable.c')
0 files changed, 0 insertions, 0 deletions