aboutsummaryrefslogtreecommitdiff
path: root/src/tutorial/funcs.source
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2007-10-24 20:54:33 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2007-10-24 20:54:33 +0000
commit72fb03b8c87acfb4c5be253f086170b8e20d7101 (patch)
treeffb4f02c983380743b52f3fa0627450d851f33c5 /src/tutorial/funcs.source
parent9632b06ec711edbf179ed64fcf4cf29f92e06967 (diff)
downloadpostgresql-72fb03b8c87acfb4c5be253f086170b8e20d7101.tar.gz
postgresql-72fb03b8c87acfb4c5be253f086170b8e20d7101.zip
Fix an error in make_outerjoininfo introduced by my patch of 30-Aug: the code
neglected to test whether an outer join's join-condition actually refers to the lower outer join it is looking at. (The comment correctly described what was supposed to happen, but the code didn't do it...) This often resulted in adding an unnecessary constraint on the join order of the two outer joins, which was bad enough. However, it also seems to expose a performance problem in an older patch (from 15-Feb): once we've decided that there is a join ordering constraint, we will start trying clauseless joins between every combination of rels within the constraint, which pointlessly eats up lots of time and space if there are numerous rels below the outer join. That probably needs to be revisited :-(. Per gripe from Jakub Ouhrabka.
Diffstat (limited to 'src/tutorial/funcs.source')
0 files changed, 0 insertions, 0 deletions