Re: parallel mode and parallel contexts

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Peter Geoghegan <pg(at)heroku(dot)com>
Cc: Andres Freund <andres(at)2ndquadrant(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: parallel mode and parallel contexts
Date: 2015-04-21 19:17:34
Message-ID: CA+TgmoartTF8eptBhiNwxUkfkctsFc7WtZFhGEGQywE8e2vCmg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Apr 20, 2015 at 6:49 PM, Peter Geoghegan <pg(at)heroku(dot)com> wrote:
> I see that you're using git format-patch to generate this. But the
> patch is only patch 1/4. Is that intentional? Where are the other
> pieces?
>
> I think that the parallel seqscan patch, and the assessing parallel
> safety patch are intended to fit together with this patch, but I can't
> find a place where there is a high level overview explaining just how
> they fit together (I notice Amit's patch has an "#include
> "access/parallel.h", which is here, but that wasn't trivial to figure
> out). I haven't been paying too much attention to this patch series.

The intended order of application is:

- parallel mode/contexts
- assess parallel safety
- parallel heap scan
- parallel seq scan

The parallel heap scan patch should probably be merged into the
parallel seq scan patch, which should probably then be split into
several patches, one or more with general parallel query
infrastructure and then another with stuff specific to parallel
sequential scan. But we're not quite there yet. Until we can get
agreement on how to finalize the parallel mode/contexts patch, I
haven't been too worried about getting the other stuff sorted out
exactly right.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2015-04-21 19:18:03 Re: PATCH: Add 'pid' column to pg_replication_slots
Previous Message Robert Haas 2015-04-21 19:14:56 Re: parallel mode and parallel contexts