Re: In core use of RegisterXactCallback() and RegisterSubXactCallback()
Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
From: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
To: Amit Langote <amitlangote09@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org
Date: 2026-06-26T04:10:57Z
Lists: pgsql-hackers
Hi, On Fri, Jun 26, 2026 at 08:21:50AM +0900, Amit Langote wrote: > On Fri, Jun 26, 2026 at 12:05 AM Bertrand Drouvot > <bertranddrouvot.pg@gmail.com> wrote: > > Just one nit: > > > > + Assert for development builds and, since > > + * the transaction is already committed by now and FK checks may have been > > + * skipped, also warn in production builds. > > > > s/development builds/assert-enabled builds/? That looks more common and " > > development builds" would be introduced by this patch. > > I grepped, and while the word "development" isn't rare in the tree, > none of the uses are "development builds" as a build type. Right. > "assert-enabled" is the established term for that, so I've switched to > it in v2. Thanks! LGTM. Regards, -- Bertrand Drouvot PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com
Commits
-
Hardwire RI fast-path end-of-xact cleanup into xact.c
- 6f4bac854fb7 19 (unreleased) landed