Re: Runtime pruning problem
Amit Langote <langote_amit_f8@lab.ntt.co.jp>
From: Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>
To: David Rowley <david.rowley@2ndquadrant.com>
Cc: Yuzuko Hosoya <hosoya.yuzuko@lab.ntt.co.jp>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2019-04-17T02:49:04Z
Lists: pgsql-hackers
On 2019/04/17 11:29, David Rowley wrote: > On Wed, 17 Apr 2019 at 13:13, Amit Langote > <Langote_Amit_f8@lab.ntt.co.jp> wrote: >> When you see this: >> >> explain select * from t1 where dt = current_date + 400; >> QUERY PLAN >> ──────────────────────────────────────────────────────────── >> Append (cost=0.00..198.42 rows=44 width=8) >> Subplans Removed: 3 >> -> Seq Scan on t1_1 (cost=0.00..49.55 rows=11 width=8) >> Filter: (dt = (CURRENT_DATE + 400)) >> (4 rows) >> >> Doesn't this give an impression that t1_1 *matches* the WHERE condition >> where it clearly doesn't? IMO, contorting explain.c to show an empty >> Append like what Hosoya-san suggests doesn't sound too bad given that the >> first reaction to seeing the above result is to think it's a bug of >> partition pruning. > > Where do you think the output list for EXPLAIN VERBOSE should put the > output column list in this case? On the Append node, or just not show > them? Maybe, not show them? That may be a bit inconsistent, because the point of VERBOSE is to the targetlist among other things, but maybe the users wouldn't mind not seeing it on such empty Append nodes. OTOH, they are more likely to think seeing a subplan that's clearly prunable as a bug of the pruning logic. Thanks, Amit
Commits
-
Put back regression test case in a more robust form.
- 22864f6e02f8 13.0 landed
-
Allow executor startup pruning to prune all child nodes.
- 5935917ce59e 13.0 landed
-
Further adjust EXPLAIN's choices of table alias names.
- 6ef77cf46e81 13.0 landed