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: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] System cache index cleanup |
Date: | 1999-11-23 04:48:16 |
Message-ID: | 199911230448.XAA14776@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > I have made most system indexes unique.
>
> Bruce, I think you need to revert the following changes to pg_opclass.h:
>
> *** src/include/catalog/pg_opclass.h 1999/09/29 21:13:30 1.20
> --- src/include/catalog/pg_opclass.h 1999/11/22 17:36:15
> ***************
> *** 68,76 ****
> DESCR("");
> DATA(insert OID = 423 ( float8_ops 701 ));
> DESCR("");
> ! DATA(insert OID = 424 ( int24_ops 0 ));
> DESCR("");
> ! DATA(insert OID = 425 ( int42_ops 0 ));
> DESCR("");
> DATA(insert OID = 426 ( int4_ops 23 ));
> DESCR("");
> --- 68,78 ----
> DESCR("");
> DATA(insert OID = 423 ( float8_ops 701 ));
> DESCR("");
> ! /* Technically, deftype is wrong, but it must be unique for index, bjm */
> ! DATA(insert OID = 424 ( int24_ops 424 ));
> DESCR("");
> ! /* Technically, deftype is wrong, but it must be unique for index, bjm */
> ! DATA(insert OID = 425 ( int42_ops 425 ));
> DESCR("");
> DATA(insert OID = 426 ( int4_ops 23 ));
> DESCR("");
> ***************
> *** 85,91 ****
> DESCR("");
> DATA(insert OID = 432 ( abstime_ops 702 ));
> DESCR("");
> ! DATA(insert OID = 433 ( bigbox_ops 603 ));
> DESCR("");
> DATA(insert OID = 434 ( poly_ops 604 ));
> DESCR("");
> --- 87,94 ----
> DESCR("");
> DATA(insert OID = 432 ( abstime_ops 702 ));
> DESCR("");
> ! /* Technically, deftype is wrong, but it must be unique for index, bjm */
> ! DATA(insert OID = 433 ( bigbox_ops 433 ));
> DESCR("");
> DATA(insert OID = 434 ( poly_ops 604 ));
> DESCR("");
>
> and make the corresponding index non-unique.
>
> (a) It is not supposed to be a unique column --- we'd not need the
> concept of index opclasses at all if there were only one possible
> operator set for any given column type!
>
> (b) The above changes are making the oidjoins and opr_sanity regress
> tests fail, as indeed they should...
Patch reverse applied, and index no longer unique. I saw these errors
too but was unsure of the cause and whether it was significant.
--
Bruce Momjian | http://www.op.net/~candle
maillist(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 | 1999-11-23 04:58:08 | Re: [HACKERS] Re: TODO updates |
Previous Message | Tom Lane | 1999-11-23 04:35:22 | Re: [HACKERS] System cache index cleanup |