On Mon, Aug 14, 2017 at 7:51 AM, Jeevan Ladhe <jeevan.la...@enterprisedb.com> wrote: > I think even with this change there will be one level of indentation > needed for throwing the error, as the error is to be thrown only if > there exists a default partition.
That's true, but we don't need two levels. >> > 0007: >> > This patch introduces code to check if the scanning of default partition >> > child >> > can be skipped if it's constraints are proven. >> >> If I understand correctly, this is actually a completely separate >> feature not intrinsically related to default partitioning. > > I don't see this as a new feature, since scanning the default partition > will be introduced by this series of patches only, and rather than a > feature this can be classified as a completeness of default skip > validation logic. Your thoughts? Currently, when a partitioned table is attached, we check whether all the scans can be checked but not whether scans on some partitions can be attached. So there are two separate things: 1. When we introduce default partitioning, we need scan the default partition either when (a) any partition is attached or (b) any partition is created. 2. In any situation where scans are needed (scanning the partition when it's attached, scanning the default partition when some other partition is attached, scanning the default when a new partition is created), we can run predicate_implied_by for each partition to see whether the scan of that partition can be skipped. Those two changes are independent. We could do (1) without doing (2) or (2) without doing (1) or we could do both. So they are separate features. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers