public inbox for [email protected]  

help / color / mirror / Atom feed

From: Alexander Pyhalov <[email protected]>
To: Justin Pryzby <[email protected]>
Cc: Masahiko Sawada <[email protected]>
Cc: Michael Paquier <[email protected]>
Cc: 李杰(慎追) <[email protected]>
Cc: [email protected]
Cc: 曾文旌(义从) <[email protected]>
Cc: Alvaro Herrera <[email protected]>
Subject: Re: CREATE INDEX CONCURRENTLY on partitioned index
Date: Wed, 29 Jun 2022 09:45:49 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<CAD21AoCHYE5=PU7kJGGUHZuYVrHTHqmpUxN+8-rLGYpdgPwJ9g@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

Justin Pryzby писал 2022-06-28 21:33:
> Hi,
> 
> On Thu, Feb 10, 2022 at 06:07:08PM +0300, Alexander Pyhalov wrote:
>> I've rebased patches and tried to fix issues I've seen. I've fixed 
>> reference
>> after table_close() in the first patch (can be seen while building 
>> with
>> CPPFLAGS='-DRELCACHE_FORCE_RELEASE').
> 
> Thanks for finding that.
> 
> The patches other than 0001 are more experimental, and need someone to 
> check if
> it's even a good approach to use, so I kept them separate from the 
> essential
> patch.
> 
> Your latest 0005 patch (mark intermediate partitioned indexes as valid) 
> is
> probably fixing a bug in my SKIPVALID patch, right ?  I'm not sure 
> whether the
> SKIPVALID patch should be merged into 0001, and I've been awaiting 
> feedback on
> the main patch before handling progress reporting.

Hi. I think it's more about fixing ReindexPartitions-to-set-indisvalid 
patch, as
we also should mark intermediate indexes as valid when reindex succeeds.


> Sorry for not responding sooner.  The patch saw no activity for ~11 
> months so I
> wasn't prepared to pick it up in March, at least not without guidance 
> from a
> committer.
> 
> Would you want to take over this patch ?  I wrote it following 
> someone's
> question, but don't expect that I'd use the feature myself.  I can help 
> review
> it or try to clarify the organization of my existing patches (but still 
> haven't
> managed to work my way through your amendments to my patches).
> 

Yes, I'm glad to work on the patches, as this for us this is a very 
important feature.
-- 
Best regards,
Alexander Pyhalov,
Postgres Professional

view thread (28+ 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], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: CREATE INDEX CONCURRENTLY on partitioned index
  In-Reply-To: <[email protected]>

* 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