Commit6d61c3f

committed

Remove unnecessary clauses_attnums variable

Commitc676e65 reworked how choose_best_statistics() picks the bestextended statistics, but failed to remove clauses_attnums which is nowunnecessary. So get rid of it and backpatch to 12, same asc676e65.Author: Tomas VondraDiscussion:https://postgr.es/m/CA+u7OA7H5rcE2=8f263w4NZD6ipO_XOrYB816nuLXbmSTH9pQQ@mail.gmail.comBackpatch-through: 12

1 parentc676e65 commit6d61c3fCopy full SHA for 6d61c3f

File tree

-4

lines changed

-4

lines changed

Lines changed: 0 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -1206,7 +1206,6 @@ statext_mcv_clauselist_selectivity(PlannerInfo root, List clauses, int varReli`
`1206`	`1206`	`RelOptInforel,Bitmapset*estimatedclauses)`
`1207`	`1207`	`{`
`1208`	`1208`	`ListCell*l;`
`1209`		`-Bitmapset*clauses_attnums=NULL;`
`1210`	`1209`	`Bitmapset**list_attnums;`
`1211`	`1210`	`intlistidx;`
`1212`	`1211`	`StatisticExtInfo*stat;`
`@@ -1244,10 +1243,7 @@ statext_mcv_clauselist_selectivity(PlannerInfo root, List clauses, int varReli`
`1244`	`1243`
`1245`	`1244`	`if (!bms_is_member(listidx,*estimatedclauses)&&`
`1246`	`1245`	`statext_is_compatible_clause(root,clause,rel->relid,&attnums))`
`1247`		`-{`
`1248`	`1246`	`list_attnums[listidx]=attnums;`
`1249`		`-clauses_attnums=bms_add_members(clauses_attnums,attnums);`
`1250`		`-}`
`1251`	`1247`	`else`
`1252`	`1248`	`list_attnums[listidx]=NULL;`
`1253`	`1249`

Comments

(0)