Re: ALTER tbl rewrite loses CLUSTER ON index

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Justin Pryzby <pryzby(at)telsasoft(dot)com>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Amit Langote <amitlangote09(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org, Justin Pryzby <pryzby(at)telsasoft(dot)com>
Subject: Re: ALTER tbl rewrite loses CLUSTER ON index
Date: 2020-03-18 02:48:37
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Tue, Mar 17, 2020 at 11:20:44AM -0500, Justin Pryzby wrote:
> On Tue, Mar 17, 2020 at 02:33:32PM +0900, Michael Paquier wrote:
>> Patch 0002 from Justin does that, I would keep this refactoring as
>> HEAD-only material though, and I don't spot any other code paths in
>> need of patching.
>> The commit message of patch 0001 is not what you wanted I guess.
> That's what git-am does, and I didn't find any option to make it less
> unreadable. I guess I should just delete the email body it inserts.


Anyway, Tom, Alvaro, are you planning to look at what is proposed on
this thread? I don't want to step on your toes if that's the case and
it seems to me that the approach taken by the patch is sound, using as
basic fix the addition of an AT_ClusterOn sub-command to the list of
commands to execute when rebuilding the table, ensuring that any
follow-up CLUSTER command will use the correct index.

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Andy Fan 2020-03-18 02:56:58 Re: [PATCH] Erase the distinctClause if the result is unique by definition
Previous Message 2020-03-18 02:48:09 Re: A bug when use get_bit() function for a long bytea string