Re: pgsql: Add support for partitioned tables and indexes in REINDEX - Mailing list pgsql-committers

From Michael Paquier
Subject Re: pgsql: Add support for partitioned tables and indexes in REINDEX
Date
Msg-id 20200908023912.GC19261@paquier.xyz
Whole thread Raw
In response to Re: pgsql: Add support for partitioned tables and indexes in REINDEX  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
On Mon, Sep 07, 2020 at 10:16:25PM -0400, Tom Lane wrote:
> Michael Paquier <michael@paquier.xyz> writes:
>> Add support for partitioned tables and indexes in REINDEX
>
> Buildfarm isn't finding the isolation test to be stable.

Yes, I have noticed the failure, and waited for more data to gather
while thinking about what I could do.  I have not come up with
anything, at least yet, so for now I have just removed the test.  The
thing was rather limited as-is as it was able just interact with the
first transaction of REINDEX building the list of partitions.
Perhaps, by using a fourth session taking a SHARE lock while another
uses at least a SHARE UPDATE EXCLUSIVE lock on one of the partitions,
it should be possible to make REINDEX wait in one of its inner
transactions for a partition.  It may be tricky to make that reliable
for the buildfarm though.
--
Michael

Attachment

pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Remove isolation test reindex-partitions
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Use return instead of exit() in configure