On 22.12.23 10:05, Alvaro Herrera wrote:
> On 2023-Nov-30, Peter Eisentraut wrote:
>
>> I noticed that when a column is dropped, RemoveAttributeById() clears out
>> certain fields in pg_attribute, but it leaves the variable-length fields at
>> the end (attacl, attoptions, and attfdwoptions) unchanged. This is probably
>> harmless, but it seems wasteful and unclean, and leaves potentially dangling
>> data lying around (for example, attacl could contain references to users
>> that are later also dropped).
>
> Yeah, this looks like an ancient oversight -- when DROP COLUMN was added
> we didn't have any varlena fields in this catalog, and when the first
> one was added (attacl in commit 3cb5d6580a33) resetting it on DROP
> COLUMN was overlooked.
>
> LGTM.
committed