Commit79d6e6a

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 parentef3fed2 commit79d6e6aCopy full SHA for 79d6e6a

File tree

-4

lines changed

-4

lines changed

Lines changed: 0 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -1059,7 +1059,6 @@ statext_mcv_clauselist_selectivity(PlannerInfo root, List clauses, int varReli`
`1059`	`1059`	`RelOptInforel,Bitmapset*estimatedclauses)`
`1060`	`1060`	`{`
`1061`	`1061`	`ListCell*l;`
`1062`		`-Bitmapset*clauses_attnums=NULL;`
`1063`	`1062`	`Bitmapset**list_attnums;`
`1064`	`1063`	`intlistidx;`
`1065`	`1064`	`StatisticExtInfo*stat;`
`@@ -1097,10 +1096,7 @@ statext_mcv_clauselist_selectivity(PlannerInfo root, List clauses, int varReli`
`1097`	`1096`
`1098`	`1097`	`if (!bms_is_member(listidx,*estimatedclauses)&&`
`1099`	`1098`	`statext_is_compatible_clause(root,clause,rel->relid,&attnums))`
`1100`		`-{`
`1101`	`1099`	`list_attnums[listidx]=attnums;`
`1102`		`-clauses_attnums=bms_add_members(clauses_attnums,attnums);`
`1103`		`-}`
`1104`	`1100`	`else`
`1105`	`1101`	`list_attnums[listidx]=NULL;`
`1106`	`1102`

Comments

(0)