Re: remove ATTRIBUTE_FIXED_PART_SIZE

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: remove ATTRIBUTE_FIXED_PART_SIZE
Date: 2018-08-18 18:37:00
Message-ID: 27705.1534617420@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> writes:
> Since the introduction of the CATALOG_VARLEN stuff, the fixed size of
> pg_attribute is exactly sizeof(FormData_pg_attribute), so the ancient
> mechanism to track the fixed size manually using
> ATTRIBUTE_FIXED_PART_SIZE can be removed.

Meh, I'm not sure about this. What about the possibility of trailing
padding after the last fixed column, as the comment you propose to
remove is talking about? Sure, we don't have that today, but we
might again the next time somebody adds a column to pg_attribute.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2018-08-18 20:22:28 Re: Fix hints on CREATE PROCEDURE errors
Previous Message Peter Eisentraut 2018-08-18 18:27:36 remove ATTRIBUTE_FIXED_PART_SIZE