On Thu, Jan 21, 2010 at 10:24 AM, Alex Hunsaker <[email protected]> wrote: > On Thu, Jan 21, 2010 at 07:30, Robert Haas <[email protected]> wrote: >> On Thu, Jan 21, 2010 at 12:57 AM, Alex Hunsaker <[email protected]> wrote: >>> Seems to me a comment about the above might be nice. Something like >>> /* Things after here are should always be default null */ in >>> pg_attribute.h ? >> >> Well... that wouldn't actually be a correct summary, so no. The point >> is that variable-length fields are not used in tuple descriptors. >> I'll add a comment about that. > > Yes that sounds much better, I was struggling to find the words. What > I was trying to express was something along the lines of you cant have > a non null value after these because you cant statically initialize > them in genbki.
Committed. Thanks for the very thorough review. ...Robert -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
