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

From: Claudio Freire <klaussfreire(at)gmail(dot)com>
To: Peter Geoghegan <pg(at)heroku(dot)com>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, 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-10 00:29:13
Message-ID: CAGTBQpa=P0jtPipSjPKiXNQpUzSOLB+Bj-TYy3FTJYeHCtYzdw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

...
On Fri, Sep 9, 2016 at 9:22 PM, Claudio Freire <klaussfreire(at)gmail(dot)com> wrote:
> Since it is true that doing so would make it impossible to keep the
> asserts about tupindex in tuplesort_heap_root_displace, I guess it
> depends on how useful those asserts are (ie: how likely it is that
> those conditions could be violated, and how damaging it could be if
> they were). If it is decided the refactor is desirable, I'd suggest
> making the common siftup producedure static inline, to allow
> tuplesort_heap_root_displace to inline and specialize it, since it
> will be called with checkIndex=False and that simplifies the resulting
> code considerably.
>
> Peter also mentioned that there were some other changes going on in
> the surrounding code that could impact this patch, so I'm marking the
> patch Waiting on Author.
>
> Overall, however, I believe the patch is in good shape. Only minor
> form issues need to be changed, the functionality seems both desirable
> and ready.

Sorry, forgot to specify, that was all about patch 3, the one about
tuplesort_heap_root_displace.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Claudio Freire 2016-09-10 00:51:08 Re: Tuplesort merge pre-reading
Previous Message Claudio Freire 2016-09-10 00:22:01 Re: Parallel tuplesort (for parallel B-Tree index creation)