public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Dmitry Koval <[email protected]>
Cc: [email protected]
Cc: Joseph Koshakow <[email protected]>
Subject: Re: Assert when executing query on partitioned table
Date: Thu, 22 Jan 2026 18:43:54 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<CAAvxfHciZfHN22y5kgXAGrd8b5Yn7JVpxJ_qFATiXEk_wV-a-w@mail.gmail.com>
	<CAAvxfHcN_zqFcY6we158+jMW3ogENAqVBOkYZ1VngvfHUijAcQ@mail.gmail.com>
	<[email protected]>
	<[email protected]>

Dmitry Koval <[email protected]> writes:
> I want to send the patch to Commitfest, so the version changed:
> [1] -> [2] (for cfbot).

I looked this over and concluded that actually, Joseph Koshakow's
proposal upthread is the correct one: just delete the faulty
Assert.  If we fall through and let the caller retry, we get
exactly the same behavior as in the non-partition-movement case,
so I'm prepared to accept that behavior as correct.  Moreover,
I guess users think so too, because this code is pretty old yet
we've heard no complaints about the production (no-Asserts)
behavior.  If we started to throw an error where we hadn't
before, I bet we *would* get complaints.

Also, I realized that we don't really need an injection point
to demonstrate the assertion failure: it's sufficient to make
the other session lock the tuple with SELECT FOR UPDATE before
we start the INSERT.  This is good because it's simpler and
it lets the test be back-patched all the way.

Pushed at 4b760a181 et al.

			regards, tom lane

view thread (2+ messages)

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]
  Subject: Re: Assert when executing query on partitioned table
  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