From: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
---|---|
To: | Dmitry Koval <d(dot)koval(at)postgrespro(dot)ru> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Add SPLIT PARTITION/MERGE PARTITIONS commands |
Date: | 2024-04-04 19:17:45 |
Message-ID: | CAPpHfduNYkMVuUSxZjD6u4tZG6aKdhgQcmhr-K9OQfeg56MNww@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi!
On Sun, Mar 31, 2024 at 5:12 AM Dmitry Koval <d(dot)koval(at)postgrespro(dot)ru> wrote:
> Patch stop applying due to changes in upstream.
> Here is a rebased version.
I've revised the patchset. Now there are two self-contained patches
coming with the documentation. Also, now each command has a paragraph
in the "Data definition" chapter. Also documentation and tests
contain geographical partitioning with all Russian cities. I think
that might create a country-centric feeling for the reader. I've
edited that to make cities spread around the world to reflect the
international spirit. Hope you're OK with this. Now, both merge and
split commands make new partitions using the parent table as the
template. And some other edits to comments, commit messages,
documentation etc.
I think this patch is well-reviewed and also has quite straightforward
implementation. The major limitation of holding ACCESS EXCLUSIVE LOCK
on the parent table is well-documented. I'm going to push this if no
objections.
------
Regards,
Alexander Korotkov
Attachment | Content-Type | Size |
---|---|---|
v30-0002-Implement-ALTER-TABLE-.-SPLIT-PARTITION-.-comman.patch | application/octet-stream | 185.1 KB |
v30-0001-Implement-ALTER-TABLE-.-MERGE-PARTITIONS-.-comma.patch | application/octet-stream | 112.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Imseih (AWS), Sami | 2024-04-04 19:20:41 | Re: Psql meta-command conninfo+ |
Previous Message | Tomas Vondra | 2024-04-04 19:11:27 | Re: incremental backup breakage in BlockRefTableEntryGetBlocks |