Re: [HACKERS] System cache index cleanup - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [HACKERS] System cache index cleanup
Date
Msg-id 199911230448.XAA14776@candle.pha.pa.us
Whole thread Raw
In response to Re: [HACKERS] System cache index cleanup  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> Bruce Momjian <pgman@candle.pha.pa.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@candle.pha.pa.us            |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] System cache index cleanup
Next
From: Tom Lane
Date:
Subject: Re: [HACKERS] Re: TODO updates