diff options
author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-03-27 12:52:50 -0300 |
---|---|---|
committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-03-27 13:14:23 -0300 |
commit | 1f171a1803c28d3ae24636c9ca3352ec82c39e5f (patch) | |
tree | 48fd0576fb3248f2cd6e740c8d7c0107fc39686a /src/backend/access/transam/commit_ts.c | |
parent | 827d6f977940952ebef4bd21fb0f97be4e20c0c4 (diff) | |
download | postgresql-1f171a1803c28d3ae24636c9ca3352ec82c39e5f.tar.gz postgresql-1f171a1803c28d3ae24636c9ca3352ec82c39e5f.zip |
Fix thinko in estimate_num_groups
The code for the reworked n-distinct estimation on commit 7b504eb282 was
written differently in a previous version of the patch, prior to commit;
on rewriting it, we missed updating an initializer. This caused the
code to (mistakenly) apply a fudge factor even in the case where a
single value is applied, leading to incorrect results.
This means that the 'relvarcount' variable name is now wrong. Add a
comment to try and make the situation clearer, and remove an incorrect
comment I added.
Problem noticed, and code patch, by Tomas Vondra. Additional commentary
by Álvaro.
Diffstat (limited to 'src/backend/access/transam/commit_ts.c')
0 files changed, 0 insertions, 0 deletions