| From: | Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> |
|---|---|
| To: | jtc331(at)gmail(dot)com |
| Cc: | David Steele <david(at)pgmasters(dot)net>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Darafei Praliaskouski <me(at)komzpa(dot)net>, Antonin Houska <ah(at)cybertec(dot)at>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Re: [HACKERS] [PATCH] Incremental sort |
| Date: | 2018-06-01 14:22:26 |
| Message-ID: | CAPpHfdsjv1+eRvoHqn1NwCEcBAVcOr0c67BFuzg50zZdHTANRg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi, James!
On Thu, May 31, 2018 at 11:10 PM James Coleman <jtc331(at)gmail(dot)com> wrote:
> I've attached an updated copy of the patch that applies cleanly to
> current master.
>
Thank you for rebasing this patch. Next time sending a patch, please make
sure you've bumped
its version, if even you made no changes there besides to pure rebase.
Otherwise, it would be
hard to distinguish patch versions, because patch files have exactly same
names.
I'd like to note, that I'm going to provide revised version of this patch
to the next commitfest.
After some conversations at PGCon regarding this patch, I got more
confident that providing
separate paths for incremental sorts is right. In the next revision of
this patch, incremental
sort paths would be provided in more cases.
------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ashutosh Bapat | 2018-06-01 14:44:43 | Re: I'd like to discuss scaleout at PGCon |
| Previous Message | Ashutosh Bapat | 2018-06-01 14:21:39 | Re: Problem while updating a foreign table pointing to a partitioned table on foreign server |