diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-02-05 01:24:55 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-02-05 01:24:55 +0000 |
commit | 244f6492614c3fdcd9b97cf0fa6ab91b263267e7 (patch) | |
tree | 25698714f91145c1484627e8af19002872a1d27e /src/backend/utils/adt/acl.c | |
parent | d1cf27a8ee7a7ac3a666f989e45f40910124756e (diff) | |
download | postgresql-244f6492614c3fdcd9b97cf0fa6ab91b263267e7.tar.gz postgresql-244f6492614c3fdcd9b97cf0fa6ab91b263267e7.zip |
Fix an old corner-case error in match_unsorted_outer(): don't consider
the cheapest-total inner path as a new candidate while truncating the
sort key list, if it already matched the full sort key list. This is
too much of a corner case to be worth back-patching, since it's unusual
for the cheapest total path to be sorted, and anyway no real harm is
done (except in JOIN_SEMI/ANTI cases where cost_mergejoin is a bit
broken at the moment). But it wasn't behaving as intended, so fix it.
Noted while examining a test case from Kevin Grittner. This error doesn't
explain his issue, but it does explain why "set enable_seqscan = off"
seemed to reproduce it for me.
Diffstat (limited to 'src/backend/utils/adt/acl.c')
0 files changed, 0 insertions, 0 deletions