diff options
| author | Tom Lane | 2011-02-18 00:00:49 +0000 |
|---|---|---|
| committer | Tom Lane | 2011-02-18 00:00:49 +0000 |
| commit | 52b60530f257b1591d8b72264cd6c0dd9aabfd46 (patch) | |
| tree | 6eda7c7cbeaac73debfe3526614a65a8db272a58 /src/backend/tsearch | |
| parent | de623f33353c96657651f9c3a6c8756616c610e4 (diff) | |
Fix tsmatchsel() to account properly for null rows.
ts_typanalyze.c computes MCE statistics as fractions of the non-null rows,
which seems fairly reasonable, and anyway changing it in released versions
wouldn't be a good idea. But then ts_selfuncs.c has to account for that.
Failure to do so results in overestimates in columns with a significant
fraction of null documents. Back-patch to 8.4 where this stuff was
introduced.
Jesper Krogh
Diffstat (limited to 'src/backend/tsearch')
| -rw-r--r-- | src/backend/tsearch/ts_selfuncs.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/backend/tsearch/ts_selfuncs.c b/src/backend/tsearch/ts_selfuncs.c index 8ce9fb46aa..7f33c16a24 100644 --- a/src/backend/tsearch/ts_selfuncs.c +++ b/src/backend/tsearch/ts_selfuncs.c @@ -189,11 +189,17 @@ tsquerysel(VariableStatData *vardata, Datum constval) /* No most-common-elements info, so do without */ selec = tsquery_opr_selec_no_stats(query); } + + /* + * MCE stats count only non-null rows, so adjust for null rows. + */ + selec *= (1.0 - stats->stanullfrac); } else { /* No stats at all, so do without */ selec = tsquery_opr_selec_no_stats(query); + /* we assume no nulls here, so no stanullfrac correction */ } return selec; |