Feedback-ID: ia2694551:Fastmail
Date: Mon, 12 Aug 2024 14:24:27 -0400
From: Alvaro Herrera from 2ndQuadrant <alvherre@alvh.no-ip.org>
To: Junwang Zhao <zhjwpku@gmail.com>
Cc: Kuntal Ghosh <kuntalghosh.2007@gmail.com>,
	Tender Wang <tndrwang@gmail.com>, pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #18559: Crash after detaching a partition concurrently from
 another session
Message-ID: <202408121824.gwvkmkugf6x7@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="kq4bryn2b3e5ezx3"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: 
 <CAEG8a3+qEsMW_R502GB9szReSGnhUvoNuvLci1wvw07+Lbegiw@mail.gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/202408121824.gwvkmkugf6x7%40alvherre.pgsql>
Precedence: bulk


--kq4bryn2b3e5ezx3
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit

On 2024-Aug-10, Junwang Zhao wrote:

> > IIUC, a simple fix would be to retry if an entry is not found. Attached a patch.
> 
> I can reproduce the issue, and the patch LGTM.

Interesting issue, thanks for reporting and putting together a
reproducer.  I have added some comments to the proposed patch, so here's
a v2 for it.  I'm going to write a commit message for it and push to all
branches since 14.

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/
"La experiencia nos dice que el hombre peló millones de veces las patatas,
pero era forzoso admitir la posibilidad de que en un caso entre millones,
las patatas pelarían al hombre" (Ijon Tichy)

--kq4bryn2b3e5ezx3
Content-Type: text/x-diff; charset=utf-8
Content-Disposition: attachment;
	filename="v2-0001-Fix-creation-of-partition-descriptor-during-concu.patch"