]> granicus.if.org Git - postgresql/commit
Fix compute_scalar_stats() for case that all values exceed WIDTH_THRESHOLD.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 11 Jan 2014 18:41:41 +0000 (13:41 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 11 Jan 2014 18:41:51 +0000 (13:41 -0500)
commit36785a21ba6003ddb846cd736170b9c194e97766
tree38c8a6df15d36ee728fd6a20b7f6086009318ff5
parenta25c2b7c4db3b4542e05d660e55bef5c93fdc32d
Fix compute_scalar_stats() for case that all values exceed WIDTH_THRESHOLD.

The standard typanalyze functions skip over values whose detoasted size
exceeds WIDTH_THRESHOLD (1024 bytes), so as to limit memory bloat during
ANALYZE.  However, we (I think I, actually :-() failed to consider the
possibility that *every* non-null value in a column is too wide.  While
compute_minimal_stats() seems to behave reasonably anyway in such a case,
compute_scalar_stats() just fell through and generated no pg_statistic
entry at all.  That's unnecessarily pessimistic: we can still produce
valid stanullfrac and stawidth values in such cases, since we do include
too-wide values in the average-width calculation.  Furthermore, since the
general assumption in this code is that too-wide values are probably all
distinct from each other, it seems reasonable to set stadistinct to -1
("all distinct").

Per complaint from Kadri Raudsepp.  This has been like this since roughly
neolithic times, so back-patch to all supported branches.
src/backend/commands/analyze.c