diff options
| author | Robert Haas <rhaas@postgresql.org> | 2017-01-13 13:29:31 -0500 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2017-01-13 13:34:10 -0500 |
| commit | 0c2070cefa0e5d097b715c9a3b9b5499470019aa (patch) | |
| tree | dd9e85fec40e2735faf51daf1a903ef52a130e57 /src/backend/executor/functions.c | |
| parent | e2117e4ab0d7fcb76f20cbf2e52541998d639d9e (diff) | |
| download | postgresql-0c2070cefa0e5d097b715c9a3b9b5499470019aa.tar.gz postgresql-0c2070cefa0e5d097b715c9a3b9b5499470019aa.zip | |
Fix cardinality estimates for parallel joins.
For a partial path, the cardinality estimate needs to reflect the
number of rows we think each worker will see, rather than the total
number of rows; otherwise, costing will go wrong. The previous coding
got this completely wrong for parallel joins.
Unfortunately, this change may destabilize plans for users of 9.6 who
have enabled parallel query, but since 9.6 is still fairly new I'm
hoping expectations won't be too settled yet. Also, this is really a
brown-paper-bag bug, so leaving it unfixed for the entire lifetime of
9.6 seems unwise.
Related reports (whose import I initially failed to recognize) by
Tomas Vondra and Tom Lane.
Discussion: http://postgr.es/m/CA+TgmoaDxZ5z5Kw_oCQoymNxNoVaTCXzPaODcOuao=CzK8dMZw@mail.gmail.com
Diffstat (limited to 'src/backend/executor/functions.c')
0 files changed, 0 insertions, 0 deletions