diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2013-07-07 22:37:24 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2013-07-07 22:37:24 -0400 |
commit | 5372275b4b5fc183c6c6dd4517cfd74d5b641446 (patch) | |
tree | 4e4927c2dc340e6206364bfc62b71eab93d913f0 /src/backend/access/transam/xlog.c | |
parent | 9b2543a4018c7f746bdb8d379c4ebc89c7d5f831 (diff) | |
download | postgresql-5372275b4b5fc183c6c6dd4517cfd74d5b641446.tar.gz postgresql-5372275b4b5fc183c6c6dd4517cfd74d5b641446.zip |
Fix planning of parameterized appendrel paths with expensive join quals.
The code in set_append_rel_pathlist() for building parameterized paths
for append relations (inheritance and UNION ALL combinations) supposed
that the cheapest regular path for a child relation would still be cheapest
when reparameterized. Which might not be the case, particularly if the
added join conditions are expensive to compute, as in a recent example from
Jeff Janes. Fix it to compare child path costs *after* reparameterizing.
We can short-circuit that if the cheapest pre-existing path is already
parameterized correctly, which seems likely to be true often enough to be
worth checking for.
Back-patch to 9.2 where parameterized paths were introduced.
Diffstat (limited to 'src/backend/access/transam/xlog.c')
0 files changed, 0 insertions, 0 deletions