Re: table AM option passing
Álvaro Herrera <alvherre@kurilemu.de>
From: Álvaro Herrera <alvherre@kurilemu.de>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: Pg Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2026-03-17T19:47:22Z
Lists: pgsql-hackers
Hi Nathan, thanks for looking, On 2026-Mar-17, Nathan Bossart wrote: > On Tue, Mar 17, 2026 at 05:50:41PM +0100, Álvaro Herrera wrote: > > (This change is vaguely similar to b7271aa1d71a, except I used 'int' > > instead of 'bits32', to keep the interface consistent with the existing > > heap_insert() one. Maybe I should make all three take bits64 instead? > > We don't actually have that type at present, so I'd have to add that > > too.) > > Why bits64 and not bits32? I must be missing something. augh, that's just a thinko -- yeah, we could use bits32 here and that wouldn't represent a reduction in number of possible flags. Does anybody oppose changing table_tuple_insert() to use bits32 instead of integer for the 'options' argument? -- Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/ "I dream about dreams about dreams", sang the nightingale under the pale moon (Sandman)
Commits
-
Give an 'options' parameter to tuple_delete/_update
- db89a47115f0 19 (unreleased) landed
-
Fix vicinity of tuple_insert to use uint32, not int, for options
- ec2f81766ad7 19 (unreleased) landed
-
Have table_insert and siblings use an unsigned type for options
- 1bd6f22f43ac 19 (unreleased) landed
-
Remove the following unused type definitions: bool8, bool16, bool32, word8,
- ca7a1f0c8684 8.0.0 cited