| 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 |
| Message-ID: | 20200318024837.GG214947@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| 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.
Strange...
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.
--
Michael
| 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 | movead.li@highgo.ca | 2020-03-18 02:48:09 | Re: A bug when use get_bit() function for a long bytea string |