Feedback-ID: i0fe9450f:Fastmail
Date: Wed, 25 Dec 2024 14:19:21 +0900
From: Michael Paquier <michael@paquier.xyz>
To: Michail Nikolaev <michail.nikolaev@gmail.com>
Cc: Matthias van de Meent <boekewurm+postgres@gmail.com>,
	PostgreSQL Hackers <pgsql-hackers@postgresql.org>,
	Andrey Borodin <amborodin86@gmail.com>,
	Melanie Plageman <melanieplageman@gmail.com>
Subject: Re: Revisiting {CREATE INDEX, REINDEX} CONCURRENTLY improvements
Message-ID: <Z2uV2TIpYisYPipp@paquier.xyz>
References: <CAEze2Wj9SgwOpe_1CWnS_D-txQaQyXArR=dm4DTnha93=yua4g@mail.gmail.com>
 <CANtu0ohFr7OzNSbxqBhUpR0mXDYyt0Xt6+=Tbq0EC7as7kr+Lg@mail.gmail.com>
 <CANtu0oh4PwBn_h+4p_MxFigRAyJvF-0nA9Tm5NFRwfsWWjZQiA@mail.gmail.com>
 <CANtu0ojHEVU9U_bxgViRmtqNTJ92LnF+76-yzn4axYjGsK2kqQ@mail.gmail.com>
 <CANtu0ogS871NkdUnZW9P_LVpLzhSJ1+cETK0b55cYjs=v2qbPA@mail.gmail.com>
 <CANtu0ohRVBDf4x7Ge3oVzgf4NzMb_DhmTM1ae0u1WUA+CD0UqA@mail.gmail.com>
 <CANtu0ogTfyng-H4yWr3Pm_+PXX+XvDx1AM1sXTy1V7DM6jJ+Bw@mail.gmail.com>
 <CANtu0oi+nbipJUsMZcoUfodCyuTN_DAXD22UstjMTYWG=tJ4jw@mail.gmail.com>
 <CANtu0oiuUF7L0wTGxOHfumyoVge3n7C4rAjdmFo=efeEwobXbg@mail.gmail.com>
 <CANtu0oiD-AvXdygYqYP-WkFq=7vSL78Wj8UU-PUX+3huPNqroQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="z+xYHpRO3nPSnOAt"
Content-Disposition: inline
In-Reply-To: <CANtu0oiD-AvXdygYqYP-WkFq=7vSL78Wj8UU-PUX+3huPNqroQ@mail.gmail.com>
Archived-At: <https://www.postgresql.org/message-id/Z2uV2TIpYisYPipp%40paquier.xyz>
Precedence: bulk


--z+xYHpRO3nPSnOAt
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Dec 24, 2024 at 02:06:26PM +0100, Michail Nikolaev wrote:
> Now STIR used for validation (but without resetting of snapshot during
> that phase for now).

Perhaps I am the only one, but what you are doing here is confusing.

There is a dependency between one patch and the follow-up ones, but
while the first patch is clear regarding its goal of improving the
interactions between REINDEX CONCURRENTLY and INSERT ON CONFLICT
regarding the selection of arbiter index in the executor in 0001 in
the scope of the other thread you have created about this problem, it
is unclear what's the goal of what you are trying to do with 0003~, if
any of the follow-up patches help with that, and even why they have a
need to be posted on this thread.  So perhaps you should split things
and explain what your goals are for each patch, or articulate better
why things are done this way?  It looks like more things just keep
piling each time a new patch series is sent to the lists.  Posting
300kB worth of patches every 3 days is not going to help potential
reviewers, just confuse them.

Note that 0002, that attempts to introduce new tests, is costly.  This
is not acceptable for integration.  I'd suggest to replace that with
tests that have controlled and successive steps as these lead to
predictible results, rather than have something that runs an arbitrary
amount of time to stress the friction of concurrent activity (this is
still useful to prove your point, though).  That's something related
to the other thread, but in passing..
--
Michael

--z+xYHpRO3nPSnOAt
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmdrldkACgkQnvQgOdby
QH01oQ//aMwQy1cg1FojT3ld9JqgvKmSDNFRxFN1qyyUnLkqCws+9OaOPs5hdW+Y
QMx3Ty+7ed17D9FTaQUdRvNz/+zmcf348KenFcajF0n9ewVTzag8+Z+NW5WYhOSc
O8q0wK98s3mCqGrK7f7rpHwslfprAWRN8aekJofuOOMrglmsQ52x7b3Tg4Co0c6A
iMGpN7xDGoCifw6CqGG/SPYru8pW0C7XZjCJsuV43wvLl+HKS1ORcbP4wZ89GjS1
7fhOOjXwQwof9KlcEeKznSlUQ5KoHkt3M607ejUfXzhc/3YKt1AmLjdEwb5uenyf
pkthMGJVr5+x6Dw31YxkUV2SRTXJ3POpiRhERY4TxUe+GCIRIQ8iPvOh2w22kLUc
Udmsd/ThjVimKTGnDIzMQ7NGZzY2nIN2XtGtFjk8167KcKEekFLf3iudh9m+Pzmj
Jtg3SkC9tQhZ/q78XBsiGfsODl9WgUTp10SmWEKfdW5uesxuoc56ZLBNaOBth7Oz
j2pN/+XLcJkAGspRLyF/gKmR/9/LahtXQBDvktMclFFEwgW8mbYUV24q8MzByZKf
awpQzx6b5AYYoNguok+FB4D1B2wcua4BbbYnc/4Offzr0/m93h40a4QQjgmbQTly
7L84MU3FL6wVOpluEGUDTGxxhHwKx9RLM/EAC3UdMkyKm0A8euQ=
=pGqx
-----END PGP SIGNATURE-----

--z+xYHpRO3nPSnOAt--