aboutsummaryrefslogtreecommitdiff
path: root/src/backend
diff options
context:
space:
mode:
authorTomas Vondra <tomas.vondra@postgresql.org>2019-11-28 23:25:14 +0100
committerTomas Vondra <tomas.vondra@postgresql.org>2019-11-28 23:25:14 +0100
commit6d61c3f1cb7134c3ad80d29e216563571cc43de2 (patch)
tree369a6b81bf1a2bf3134145a2c935566d34a0a0ff /src/backend
parentc676e659b246f94d571b57b559f80cb2dc03e73b (diff)
downloadpostgresql-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.c4
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;