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:25:14 +0100 |
commit | 6d61c3f1cb7134c3ad80d29e216563571cc43de2 (patch) | |
tree | 369a6b81bf1a2bf3134145a2c935566d34a0a0ff /src/backend | |
parent | c676e659b246f94d571b57b559f80cb2dc03e73b (diff) | |
download | postgresql-6d61c3f1cb7134c3ad80d29e216563571cc43de2.tar.gz postgresql-6d61c3f1cb7134c3ad80d29e216563571cc43de2.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/backend')
-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 6299011ca66..9d339433f65 100644 --- a/src/backend/statistics/extended_stats.c +++ b/src/backend/statistics/extended_stats.c @@ -1206,7 +1206,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; @@ -1244,10 +1243,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; |