public inbox for [email protected]
help / color / mirror / Atom feedFrom: Sami Imseih <[email protected]>
To: Michael Paquier <[email protected]>
Cc: Haibo Yan <[email protected]>
Cc: Mohamed ALi <[email protected]>
Cc: [email protected]
Subject: Re: [PATCH] Fix: Partitioned parent index remains invalid after child indexes are repaired
Date: Wed, 22 Apr 2026 05:33:09 -0500
Message-ID: <CAA5RZ0sQfnLyv2Ta-r6y==QwwUr+M0BxVbK0=0_HZZA1ywc3Bw@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAGnOmWqi1D9ycBgUeOGf6mOCd2Dcf=6sKhbf4sHLs5xAcKVCMQ@mail.gmail.com>
<CABXr29GbJe7-vbm0Fn=HWx3vQonwFtv_zO1pSc5FZAvjLuUDVA@mail.gmail.com>
<CABXr29Epj=23eYAWg_mTdiXYQQzJSVN22uk6tK8ei0ts5wXvSQ@mail.gmail.com>
<CAA5RZ0s16eHrzcDupipAxC++e-p_n_M7Fpg-MG-V59fB=wykLw@mail.gmail.com>
<[email protected]>
<[email protected]>
<CAA5RZ0uM7_YPdSahGh1sGeewMb9SzKswyLaZty8aR+a7XpQydQ@mail.gmail.com>
<[email protected]>
<[email protected]>
> > One thing that I'm tempted to add is more scans to check indisvalid
> > across these commands, particularly after the individual ATTACH
> > PARTITION bits on each individual index.
That works.
> > A second thing. Do you think that it would be worth adding a
> > partitioned table that has no leaves in some portion of the test? I
> > was thinking about a partitioned table called idxpart2 attached to
> > idxpart in the first part of the test. I've found this pattern
> > usually useful for this area of the code when recursing with
> > validatePartitionedIndex() from a parent.
Good idea.
> Both things have been added to the tests, and applied the result down
> to v14. The patch was able to apply cleanly across the board, without
> conflicts. That's rare, these days..
Sorry for the late reply, and thanks for getting this committed!
--
Sami Imseih
Amazon Web Services (AWS)
view thread (12+ 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]
Subject: Re: [PATCH] Fix: Partitioned parent index remains invalid after child indexes are repaired
In-Reply-To: <CAA5RZ0sQfnLyv2Ta-r6y==QwwUr+M0BxVbK0=0_HZZA1ywc3Bw@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