Fix thinko in estimate_num_groups
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 27 Mar 2017 15:52:50 +0000 (12:52 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 27 Mar 2017 16:14:23 +0000 (13:14 -0300)
commit1f171a1803c28d3ae24636c9ca3352ec82c39e5f
tree48fd0576fb3248f2cd6e740c8d7c0107fc39686a
parent827d6f977940952ebef4bd21fb0f97be4e20c0c4
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.
src/backend/utils/adt/selfuncs.c