| From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Joe Mitchell <jmitchell(at)greatbridge(dot)com>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Duplicate OIDs in pg_attribute |
| Date: | 2001-02-07 00:52:27 |
| Message-ID: | 200102070052.TAA24669@candle.pha.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> Good catch, Joe! This bug has probably been there since the beginning
> of time. It's evidently got no serious consequences (since in reality,
> OID uniqueness is not assumed for this table), but it ought to be fixed.
> A quick-hack solution would be to zero out the tuple's OID before each
> heap_insert, but really AppendAttributeTuples should be rewritten to
> construct each tuple independently in the first place.
>
> A quick 'glimpse' shows no other uses of heap_modifytuple except to
> update an existing tuple, so evidently no one was foolish enough to
> copy this technique.
>
> I recommend putting this on the TODO for 7.2. We can't fix it now
> unless we want to force an initdb.
Added to TODO:
* Prevent pg_attribute from having duplicate oids for indexes
--
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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2001-02-07 00:56:37 | Re: Include files for SPI are not installed |
| Previous Message | Tom Lane | 2001-02-07 00:51:06 | Re: OID from insert has extra letter |