Fix an old corner-case error in match_unsorted_outer(): don't consider
commit4f4e7f37412a30aa584387b951a3d3323836353d
authortgl <tgl>
Thu, 5 Feb 2009 01:24:55 +0000 (5 01:24 +0000)
committertgl <tgl>
Thu, 5 Feb 2009 01:24:55 +0000 (5 01:24 +0000)
tree377ddd138423e5266a5f24602dba6fc567c6e35a
parent2d45519daf59080d626313d08bf4565a631cb293
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.
src/backend/optimizer/path/joinpath.c