Re: DROP COLUMN

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: "Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)jp>, "Bruce Momjian" <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: "Christopher Kings-Lynne" <chriskl(at)familyhealth(dot)com(dot)au>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Rod Taylor" <rbt(at)zort(dot)ca>, "PostgreSQL-development" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: DROP COLUMN
Date: 2002-07-17 07:36:54
Message-ID: D85C66DA59BA044EB96AB9683819CF6101513E@dogbert.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> -----Original Message-----
> From: Hiroshi Inoue [mailto:Inoue(at)tpf(dot)co(dot)jp]
> Sent: 17 July 2002 05:12
> To: Bruce Momjian
> Cc: Christopher Kings-Lynne; Tom Lane; Rod Taylor;
> PostgreSQL-development
> Subject: Re: [HACKERS] DROP COLUMN
>
>
> > >From my perspective, when client coders like Dave Page and
> others say
> > they would prefer the flag to the negative attno's, I don't have to
> > understand. I just take their word for it.
>
> do they really love to check attisdropped everywhere ?
> Isn't it the opposite of the encapsulation ?
> I don't understand why we would do nothing for clients.

In pgAdmin's case, this involves one test (maybe 3 lines of code),
because all access to column info is made through one class. The reason
I voted for attisdropped is that the negative attnum's are assumed by
pgAdmin to be 'system columns', not 'any column that doesn't belong to
the user'. Coding around a change like that - whilst not necessarily
harder - would certainly be messier.

Regards, Dave.

Browse pgsql-hackers by date

  From Date Subject
Next Message Jan Wieck 2002-07-17 07:56:55 Re: pg_views.definition
Previous Message Hiroshi Inoue 2002-07-17 06:48:42 Re: DROP COLUMN