| From: | Dmitry Koval <d(dot)koval(at)postgrespro(dot)ru> |
|---|---|
| To: | jian he <jian(dot)universality(at)gmail(dot)com> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Add SPLIT PARTITION/MERGE PARTITIONS commands |
| Date: | 2025-06-24 21:28:28 |
| Message-ID: | da035c5a-bd58-45ea-938d-71857c981147@postgrespro.ru |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi!
Thanks for notes!
1.
>here, we don't need ``(void *)``
Corrected.
2.
>In the synopsis section, we can combine the last two lines into one
>for better formatting.
Changed.
3.
> after ...
>we can add the following to briefly explain parameters: partition_name1,
>partition_name2
Added.
4.
>What do you think about alternative syntax:
>ALTER TABLE tab1 MERGE PARTITION part1 WITH (part2, part3) mentioned
in [1].
>....
Is it additional syntax (to the existing one) with functionality:
partition part1 survives and data from partitions part2, part3 is moved
into part1?
And (if "yes") should we delete the part1 indexes (or other constraints)
before moving the data?
[1]
https://postgr.es/m/CA+TgmoY0=bT_xBP8csR=MFE=FxGE2n2-me2-31jBOgEcLvW7ug@mail.gmail.com
--
With best regards,
Dmitry Koval
Postgres Professional: http://postgrespro.com
| Attachment | Content-Type | Size |
|---|---|---|
| v49-0001-Implement-ALTER-TABLE-.-MERGE-PARTITIONS-.-comma.patch | text/plain | 165.8 KB |
| v49-0002-Implement-ALTER-TABLE-.-SPLIT-PARTITION-.-comman.patch | text/plain | 219.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jacob Champion | 2025-06-24 21:28:40 | Re: [PATCH] Fix OAuth feature detection on OpenBSD+Meson |
| Previous Message | Tom Lane | 2025-06-24 21:03:28 | Re: [PATCH] Fix OAuth feature detection on OpenBSD+Meson |