public inbox for [email protected]  

help / color / mirror / Atom feed

From: Alexander Korotkov <[email protected]>
To: Dmitry Koval <[email protected]>
Cc: stephane tachoires <[email protected]>
Cc: [email protected]
Subject: Re: Add SPLIT PARTITION/MERGE PARTITIONS commands
Date: Fri, 31 Jan 2025 11:07:17 +0200
Message-ID: <CAPpHfdvVMdUX0DGSK3oAbt9C5TPup=BEq8QkmvDrMbuD4BR9Fw@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

Hi, Dmitry!

On Tue, Jan 28, 2025 at 1:15 AM Dmitry Koval <[email protected]> wrote:
>
>  >Only patches v34-0001 and 2 has been tested.
>  >Patch v34-0003-Refactor-createPartitionTable-to-remove-ProcessU.patch
>  >do not apply anymore on src/backend/catalog/heap.c
>
> Thanks, rebased.
> The patches are attached to the email.

Thank you for the rebase.
I don't think we need a separate 0003 patch with refactoring.  It's
probably good idea to keep this functionality as a separate patch, but
let's make then it a 0001, which prepares functions used by 0002 and
0003.

------
Regards,
Alexander Korotkov
Supabase

view thread (61+ 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: <CAPpHfdvVMdUX0DGSK3oAbt9C5TPup=BEq8QkmvDrMbuD4BR9Fw@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