public inbox for [email protected]
help / color / mirror / Atom feedFrom: Zhihong Yu <[email protected]>
To: Dmitry Koval <[email protected]>
Cc: Matthias van de Meent <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Add SPLIT PARTITION/MERGE PARTITIONS commands
Date: Tue, 31 May 2022 15:14:25 -0700
Message-ID: <CALNJ-vRsNi5bNy2M8BU+oPOimqm5J2ZU6HK_w1SnhGu8mT99yg@mail.gmail.com> (raw)
In-Reply-To: <CALNJ-vSMinSB=BTPj4zthMAV3N_1t6vO2rVYBt4uiz3dif-_DA@mail.gmail.com>
References: <[email protected]>
<[email protected]>
<CAEze2WhU9Wkn2gD9FC6kbuh1gUdqCmdi9_d62-NeBkPZ_5=eOA@mail.gmail.com>
<[email protected]>
<CALNJ-vSMinSB=BTPj4zthMAV3N_1t6vO2rVYBt4uiz3dif-_DA@mail.gmail.com>
On Tue, May 31, 2022 at 1:43 PM Zhihong Yu <[email protected]> wrote:
>
>
> On Tue, May 31, 2022 at 12:43 PM Dmitry Koval <[email protected]>
> wrote:
>
>> >Just out of curiosity, why is SPLIT / MERGE support not included for
>> >HASH partitions? Because sibling partitions can have a different
>> >modulus, you should be able to e.g. split a partition with (modulus,
>> >remainder) of (3, 1) into two partitions with (mod, rem) of (6, 1) and
>> >(6, 4) respectively, with the reverse being true for merge operations,
>> >right?
>>
>> You are right, SPLIT/MERGE operations can be added for HASH-partitioning
>> in the future. But HASH-partitioning is rarer than RANGE- and
>> LIST-partitioning and I decided to skip it in the first step.
>> Maybe community will say that SPLIT/MERGE commands are not needed... (At
>> first step I would like to make sure that it is no true)
>>
>> P.S. I attached patch with 1-line warning fix (for cfbot).
>> --
>> With best regards,
>> Dmitry Koval
>>
>> Postgres Professional: http://postgrespro.com
>
>
> Hi,
> For attachPartTable, the parameter wqueue is missing from comment.
> The parameters of CloneRowTriggersToPartition are called parent
> and partition. I think it is better to name the parameters to
> attachPartTable in a similar manner.
>
> For struct SplitPartContext, SplitPartitionContext would be better name.
>
> + /* Store partition contect into list. */
> contect -> context
>
> Cheers
>
Hi,
For transformPartitionCmdForMerge(), nested loop is used to detect
duplicate names.
If the number of partitions in partcmd->partlist, we should utilize map to
speed up the check.
For check_parent_values_in_new_partitions():
+ if (!find_value_in_new_partitions(&key->partsupfunc[0],
+ key->partcollation, parts,
nparts, datum, false))
+ found = false;
It seems we can break out of the loop when found is false.
Cheers
view thread (163+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: Add SPLIT PARTITION/MERGE PARTITIONS commands
In-Reply-To: <CALNJ-vRsNi5bNy2M8BU+oPOimqm5J2ZU6HK_w1SnhGu8mT99yg@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox