Re: Asynchronous Append on postgres_fdw nodes.
Andrei Lepikhov <a.lepikhov@postgrespro.ru>
From: Andrey Lepikhov <a.lepikhov@postgrespro.ru>
To: Etsuro Fujita <etsuro.fujita@gmail.com>
Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>,
Justin Pryzby <pryzby@telsasoft.com>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-05-07T10:35:50Z
Lists: pgsql-hackers
On 6/5/21 14:11, Etsuro Fujita wrote: > On Tue, Apr 27, 2021 at 3:57 PM Andrey V. Lepikhov > <a.lepikhov@postgrespro.ru> wrote: >> One more question. Append choose async plans at the stage of the Append >> plan creation. >> Later, the planner performs some optimizations, such as eliminating >> trivial Subquery nodes. So, AsyncAppend is impossible in some >> situations, for example: >> >> (SELECT * FROM f1 WHERE a < 10) >> UNION ALL >> (SELECT * FROM f2 WHERE a < 10); >> >> But works for the query: >> >> SELECT * >> FROM (SELECT * FROM f1 UNION ALL SELECT * FROM f2) AS q1 >> WHERE a < 10; >> >> As far as I understand, this is not a hard limit. > > I think so, but IMO I think this would be an improvement rather than a bug fix. > >> We can choose async >> subplans at the beginning of the execution stage. >> For a demo, I prepared the patch (see in attachment). >> It solves the problem and passes the regression tests. > > Thanks for the patch! IIUC, another approach to this would be the > patch you proposed before [1]. Right? Yes. I think, new solution will be better. -- regards, Andrey Lepikhov Postgres Professional
Commits
-
Improve comments for trivial_subqueryscan().
- 4a8a5dd7f59c 15.0 landed
-
Disable asynchronous execution if using gating Result nodes.
- ebb790241523 14.3 landed
- 5c854e7a2c8a 15.0 landed
-
Allow asynchronous execution in more cases.
- c2bb02bc2e85 15.0 landed
-
Doc: Further update documentation for asynchronous execution.
- eab81953682d 14.0 landed
-
Fix rescanning of async-aware Append nodes.
- f3baaf28a6da 14.0 landed
-
Doc: Update documentation for asynchronous execution.
- 15fcd33e0694 14.0 landed
-
Fix EXPLAIN ANALYZE for async-capable nodes.
- a363bc6da96b 14.0 landed
-
Minor code cleanup in asynchronous execution support.
- bb684c82f733 14.0 landed
-
Adjust input value to WaitEventSetWait() in ExecAppendAsyncEventWait().
- a8af856d3257 14.0 landed
-
Add support for asynchronous execution.
- 27e1f14563cf 14.0 landed