Re: Use CompactAttribute more often, when possible - Mailing list pgsql-hackers

From David Rowley
Subject Re: Use CompactAttribute more often, when possible
Date
Msg-id CAApHDvpyzwBhZEa68pwu3YHDhjAEC2pSMCvtsE2CyjcW7=r6Cg@mail.gmail.com
Whole thread Raw
In response to Re: Use CompactAttribute more often, when possible  (David Rowley <dgrowleyml@gmail.com>)
List pgsql-hackers
On Mon, 20 Oct 2025 at 21:43, David Rowley <dgrowleyml@gmail.com> wrote:
>
> On Mon, 20 Oct 2025 at 21:15, Michael Paquier <michael@paquier.xyz> wrote:
> >
> > On Mon, Oct 20, 2025 at 05:46:33PM +1300, David Rowley wrote:
> > @@ -5146,10 +5146,6 @@ ReorderBufferToastReplace(ReorderBuffer *rb, ReorderBufferTXN *txn,
> >          dlist_iter    it;
> >          Size        data_done = 0;
> >
> > -        /* system columns aren't toasted */
> > -        if (attr->attnum < 0)
> > -            continue;
> >
> > Er, why this removal?

> We don't put system attributes in TupleDescs so I put that check down
> to misguided programming.

Thank you for reviewing this. Now pushed.

David



pgsql-hackers by date:

Previous
From: Jim Jones
Date:
Subject: Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREATE TRIGGER statement
Next
From: Peter Smith
Date:
Subject: Re: POC: enable logical decoding when wal_level = 'replica' without a server restart