| From: | Surafel Temesgen <surafel3000(at)gmail(dot)com> |
|---|---|
| To: | Alexey Kondratov <a(dot)kondratov(at)postgrespro(dot)ru> |
| Cc: | Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Allow CLUSTER, VACUUM FULL and REINDEX to change tablespace on the fly |
| Date: | 2019-09-17 11:04:37 |
| Message-ID: | CALAY4q-0oU4ORb07C4BxvGXgetBswqS1V5e2JP1wTV5q+O7-tA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Alexey
Here are a few comment
On Sat, Aug 31, 2019 at 11:54 PM <a(dot)kondratov(at)postgrespro(dot)ru> wrote:
> Hi hackers,
>
>
> Anyway, new patch, which adds SET TABLESPACE to REINDEX is attached and
> this functionality seems really useful, so I will be very appreciate if
> someone will take a look on it.
>
* There are NOWAIT option in alter index, is there a reason not to have
similar option here?
* SET TABLESPACE command is not documented
* There are multiple checking for whether the relation is temporary tables
of other sessions, one in check_relation_is_movable and other independently
*+ char *tablespacename;
calling it new_tablespacename will make it consistent with other places
*The patch did't applied cleanly http://cfbot.cputube.org/patch_24_2269.log
regards
Surafel
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ashutosh Sharma | 2019-09-17 11:15:11 | Re: Zedstore - compressed in-core columnar storage |
| Previous Message | Amit Kapila | 2019-09-17 10:54:21 | Re: pgbench - allow to create partitioned tables |