scan on inheritance parent with no children in current session
Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
From: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
To: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2017-08-04T09:11:38Z
Lists: pgsql-hackers
Attachments
- no_child_parent_scan.patch (text/x-patch) patch
Hi All,
Consider a parent table which has no child in the current session, but
has temporary children in other sessions.
Session 1
postgres=# create table parent (a int);
CREATE TABLE
Session 2:
postgres=# create temp table temp_child () inherits(parent);
CREATE TABLE
Before commit d3cc37f1d801a6b5cad9bf179274a8d767f1ee50. We would have
Seq Scan plan for scanning parent in session 1
postgres=# explain verbose select * from parent;
QUERY PLAN
-------------------------------------------------------------
Seq Scan on public.parent (cost=0.00..0.00 rows=1 width=4)
Output: parent.a
(2 rows)
In session 2, it would be an Append plan
postgres=# explain verbose select * from parent;
QUERY PLAN
------------------------------------------------------------------------------
Append (cost=0.00..35.50 rows=2551 width=4)
-> Seq Scan on public.parent (cost=0.00..0.00 rows=1 width=4)
Output: parent.a
-> Seq Scan on pg_temp_4.temp_child (cost=0.00..35.50 rows=2550 width=4)
Output: temp_child.a
(5 rows)
After that commit in session 1, we get an Append plan
postgres=# explain verbose select * from parent;
QUERY PLAN
-------------------------------------------------------------------
Append (cost=0.00..0.00 rows=1 width=4)
-> Seq Scan on public.parent (cost=0.00..0.00 rows=1 width=4)
Output: parent.a
(3 rows)
I don't think this is an intentional change. Here's patch to fix it.
The comment in the patch uses term "real child" in the context of
comments about temporary children from other session and the comment
at the end of the function where rte->inh is reset. May be we should
move the second comment before setting has_child in the patch and use
"real child" in the comment at the end to avoid repetition. But I want
to first check whether we want this fix or we can live with the Append
plan.
--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company
Commits
-
Avoid unnecessary single-child Append nodes.
- 17a2a27b49ae 10.0 landed
- d57929afc706 11.0 landed
-
Don't scan partitioned tables.
- d3cc37f1d801 10.0 cited