Re: Unused pg_class columns

> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Pgaccess is referencing system columns that are never set.
>
> I wasn't aware that our standard procedure upon finding a bug was to rip
> out the feature, rather than fixing the bug ... this could make life a
> *lot* simpler.

Attached is a patch of all my pgaccess changes this morning. If someone
wants to code the hasprimary test, feel free. You can look at psql to
see new code that does tests for primary keys. It was recently
committed to CVS.

If people would prefer me to comment out the primary key display rather
than remove it, I can do that.

> If relhaspkey isn't set by table creation, then let's set it. Easy
> enough, and it seems like a clearly useful column. As for the other
> columns at issue, those are not ancient history: they were added in
> release 6.4, by Vadim according to the CVS logs. Perhaps you should ask
> him what they are intended for, rather than assuming that they are fair
> game for removal.

I only marked as "not used", not as "remove me". Seems clear enough.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026