diff options
| author | Tomas Vondra <tomas.vondra@postgresql.org> | 2019-11-28 23:25:14 +0100 |
|---|---|---|
| committer | Tomas Vondra <tomas.vondra@postgresql.org> | 2019-11-28 23:28:53 +0100 |
| commit | 79d6e6afabcb3766ea4d21e3e26d25a5f2f69553 (patch) | |
| tree | 4632ecf5b29a93d579b089f03fb72a78ae43c2c4 /src | |
| parent | ef3fed2ce4a3018a992ef913a3333bb682b702ae (diff) | |
| download | postgresql-79d6e6afabcb3766ea4d21e3e26d25a5f2f69553.tar.gz postgresql-79d6e6afabcb3766ea4d21e3e26d25a5f2f69553.zip | |
Remove unnecessary clauses_attnums variable
Commit c676e659b2 reworked how choose_best_statistics() picks the best
extended statistics, but failed to remove clauses_attnums which is now
unnecessary. So get rid of it and backpatch to 12, same as c676e659b2.
Author: Tomas Vondra
Discussion: https://postgr.es/m/CA+u7OA7H5rcE2=8f263w4NZD6ipO_XOrYB816nuLXbmSTH9pQQ@mail.gmail.com
Backpatch-through: 12
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/statistics/extended_stats.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/backend/statistics/extended_stats.c b/src/backend/statistics/extended_stats.c index ca6d5c34e36..4516c97f406 100644 --- a/src/backend/statistics/extended_stats.c +++ b/src/backend/statistics/extended_stats.c @@ -1059,7 +1059,6 @@ statext_mcv_clauselist_selectivity(PlannerInfo *root, List *clauses, int varReli RelOptInfo *rel, Bitmapset **estimatedclauses) { ListCell *l; - Bitmapset *clauses_attnums = NULL; Bitmapset **list_attnums; int listidx; StatisticExtInfo *stat; @@ -1097,10 +1096,7 @@ statext_mcv_clauselist_selectivity(PlannerInfo *root, List *clauses, int varReli if (!bms_is_member(listidx, *estimatedclauses) && statext_is_compatible_clause(root, clause, rel->relid, &attnums)) - { list_attnums[listidx] = attnums; - clauses_attnums = bms_add_members(clauses_attnums, attnums); - } else list_attnums[listidx] = NULL; |