NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit3b45944

committed

Remove unnecessary code in dependency_is_compatible_expression().

Scanning the expression for compatible Vars isn't really necessary,because the subsequent match against StatisticExtInfo entries willeliminate expressions containing other Vars just fine. Moreover,this code hadn't stopped to think about what to do withPlaceHolderVars or Aggrefs in the clause; and at least for the PHVcase, that demonstrably leads to failures. Rather than work outwhether it's reasonable to ignore those, let's just remove thewhole stanza.Per report from Richard Guo. Back-patch to v14 where this codewas added.Discussion:https://postgr.es/m/CAMbWs48Mmvm-acGevXuwpB=g5JMqVSL6i9z5UaJyLGJqa-XPAA@mail.gmail.com

1 parent74a1a36 commit3b45944Copy full SHA for 3b45944

File tree

1 file changed

-25

lines changed

src/backend/statistics
- dependencies.c

1 file changed

-25

lines changed

`‎src/backend/statistics/dependencies.c`

Lines changed: 3 additions & 25 deletions

Original file line number	Diff line number	Diff line change
`@@ -1163,13 +1163,12 @@ clauselist_apply_dependencies(PlannerInfo root, List clauses,`
`1163`	`1163`	`*Determines if the expression is compatible with functional dependencies`
`1164`	`1164`	`*`
`1165`	`1165`	`* Similar to dependency_is_compatible_clause, but doesn't enforce that the`
`1166`		`- * expression is a simple Var.OTOH we check that there's at least one`
`1167`		`- *statistics object matching the expression.`
	`1166`	`+ * expression is a simple Var. On success, return the matching statistics`
	`1167`	`+ expression into expr.`
`1168`	`1168`	`*/`
`1169`	`1169`	`staticbool`
`1170`	`1170`	`dependency_is_compatible_expression(Nodeclause,Indexrelid,Liststatlist,Node**expr)`
`1171`	`1171`	`{`
`1172`		`-List*vars;`
`1173`	`1172`	`ListCell*lc,`
`1174`	`1173`	`*lc2;`
`1175`	`1174`	`Node*clause_expr;`
`@@ -1317,29 +1316,8 @@ dependency_is_compatible_expression(Node clause, Index relid, List statlist, N`
`1317`	`1316`	`if (IsA(clause_expr,RelabelType))`
`1318`	`1317`	`clause_expr= (Node) ((RelabelType)clause_expr)->arg;`
`1319`	`1318`
`1320`		`-vars=pull_var_clause(clause_expr,0);`
`1321`		`-`
`1322`		`-foreach(lc,vars)`
`1323`		`-{`
`1324`		`-Varvar= (Var)lfirst(lc);`
`1325`		`-`
`1326`		`-/* Ensure Var is from the correct relation */`
`1327`		`-if (var->varno!=relid)`
`1328`		`-return false;`
`1329`		`-`
`1330`		`-/* We also better ensure the Var is from the current level */`
`1331`		`-if (var->varlevelsup!=0)`
`1332`		`-return false;`
`1333`		`-`
`1334`		`-/* Also ignore system attributes (we don't allow stats on those) */`
`1335`		`-if (!AttrNumberIsForUserDefinedAttr(var->varattno))`
`1336`		`-return false;`
`1337`		`-}`
`1338`		`-`
`1339`	`1319`	`/*`
`1340`		`- * Check if we actually have a matching statistics for the expression.`
`1341`		`- *`
`1342`		`- * XXX Maybe this is an overkill. We'll eliminate the expressions later.`
	`1320`	`+ * Search for a matching statistics expression.`
`1343`	`1321`	`*/`
`1344`	`1322`	`foreach(lc,statlist)`
`1345`	`1323`	`{`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit3b45944

File tree

1 file changed

1 file changed

`‎src/backend/statistics/dependencies.c`

0 commit comments