Re: Parallel tuplesort (for parallel B-Tree index creation)

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Peter Geoghegan <pg(at)heroku(dot)com>
Cc: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Corey Huinker <corey(dot)huinker(at)gmail(dot)com>
Subject: Re: Parallel tuplesort (for parallel B-Tree index creation)
Date: 2016-09-27 14:08:14
Message-ID: CA+TgmoZ4vi1d6fq4Zct=PgvJjR_8ktj5iuGbCXSfZGYjh5uFOw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Sep 26, 2016 at 3:40 PM, Peter Geoghegan <pg(at)heroku(dot)com> wrote:
> On Mon, Sep 26, 2016 at 6:58 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>> That requires some kind of mutual exclusion mechanism, like an LWLock.
>>
>> No, it doesn't. Shared memory queues are single-reader, single-writer.
>
> The point is that there is a natural dependency when merging is
> performed eagerly within the leader. One thing needs to be in lockstep
> with the others. That's all.

I don't know what any of that means. You said we need something like
an LWLock, but I think we don't. The workers just write the results
of their own final merges into shm_mqs. The leader can read from any
given shm_mq until no more tuples can be read without blocking, just
like nodeGather.c does, or at least it can do that unless its own
queue fills up first. No mutual exclusion mechanism is required for
any of that, as far as I can see - not an LWLock, and not anything
similar.

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jesper Pedersen 2016-09-27 14:10:42 Re: pageinspect: Hash index support
Previous Message Peter Geoghegan 2016-09-27 14:00:40 Re: Refactoring speculative insertion with unique indexes a little