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.
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) |