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 |