Re: pgsql: Support Parallel Append plan nodes.
Amit Khandekar <amitdkhan.pg@gmail.com>
From: Amit Khandekar <amitdkhan.pg@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>, amul sul <sulamul@gmail.com>, pgsql-committers <pgsql-committers@postgresql.org>
Date: 2017-12-07T07:02:46Z
Lists: pgsql-hackers
On 6 December 2017 at 20:59, Tom Lane <tgl@sss.pgh.pa.us> wrote: > Robert Haas <robertmhaas@gmail.com> writes: >> On Wed, Dec 6, 2017 at 5:01 AM, Amit Khandekar <amitdkhan.pg@gmail.com> wrote: >>> In attached revised patch, just added some comments in the changes that you did. > >> Committed, thanks. > > While this patch (looks like it) fixes the logic error, it does nothing > for the problem that the committed test cases don't actually exercise > any of this code on most machines -- certainly not whichever one is > producing the code coverage report: > https://coverage.postgresql.org/src/backend/executor/nodeAppend.c.gcov.html > > Can we do something with Andres' suggestion of running these test cases > under parallel_leader_participation = off? > > regards, tom lane Yes, I am planning to send a patch that makes all those Parallel-Append test cases run once with parallel_leader_participation "on" and then run again with the guc "off" . Thanks. -- Thanks, -Amit Khandekar EnterpriseDB Corporation The Postgres Database Company
Commits
-
Fix Parallel Append crash.
- 9c64ddd41485 11.0 landed
-
Support Parallel Append plan nodes.
- ab7271677812 11.0 cited