Hi David,
On Tue, Feb 24, 2026 at 11:23 AM David Rowley <[email protected]> wrote:
> I've attached an updated version of the patch (v9).
I noticed what looks like some rebase noise in TupleDescFinalize():
off = att_nominal_alignby(off, cattr->attalignby);
/*
* attcacheoff is an int16, so don't try and cache any offsets larger
* than will fit in that type.
*/
if (off > PG_INT16_MAX)
break;
cattr->attcacheoff = off;
/*
* attcacheoff is an int16, so don't try and cache any offsets larger
* than will fit in that type.
*/
if (off > PG_INT16_MAX)
break;
cattr->attcacheoff = off;
--
Thanks, Amit Langote