Commit1cbc17a

committed

Try to exclude partitioned tables in toto.

1 parent59c8078 commit1cbc17aCopy full SHA for 1cbc17a

File tree

+12

-2

lines changed

+12

-2

lines changed

Lines changed: 12 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1414,8 +1414,18 @@ relation_excluded_by_constraints(PlannerInfo *root,`
`1414`	`1414`	`if (predicate_refuted_by(safe_restrictions,safe_restrictions, false))`
`1415`	`1415`	`return true;`
`1416`	`1416`
`1417`		`-/* Only plain relations have constraints */`
`1418`		`-if (rte->rtekind!=RTE_RELATION\|\|rte->inh)`
	`1417`	`+/*`
	`1418`	`+ * Only plain relations have constraints. In a partitioning hierarchy,`
	`1419`	`+ * but not with regular table inheritance, it's OK to assume that any`
	`1420`	`+ * constraints that hold for the parent also hold for every child; for`
	`1421`	`+ * instance, table inheritance allows the parent to have constraints`
	`1422`	`+ * marked NO INHERIT, but table partitioning does not. We choose to check`
	`1423`	`+ * whether the partitioning parents can be excluded here; doing so`
	`1424`	`+ * consumes some cycles, but potentially saves us the work of excluding`
	`1425`	`+ * each child individually.`
	`1426`	`+ */`
	`1427`	`+if (rte->rtekind!=RTE_RELATION\|\|`
	`1428`	`+(rte->inh&&rte->relkind!=RELKIND_PARTITIONED_TABLE))`
`1419`	`1429`	`return false;`
`1420`	`1430`
`1421`	`1431`	`/*`

Comments

(0)