Thread: Small FK patch to deal with tables without oids

Small FK patch to deal with tables without oids

From
Stephan Szabo
Date:
    Tables without oids wouldn't be able to be
used inside fk constraints, since some of the checks
in the trigger did a SELECT oid.  Since the oid wasn't
actually used, I changed this to SELECT 1.  My test
case with non-oid tables now works and fk regression
appears to run fine on my machine.

Attachment

Re: Small FK patch to deal with tables without oids

From
Tom Lane
Date:
Stephan Szabo <sszabo@megazone23.bigpanda.com> writes:
>     Tables without oids wouldn't be able to be
> used inside fk constraints, since some of the checks
> in the trigger did a SELECT oid.  Since the oid wasn't
> actually used, I changed this to SELECT 1.

Can't believe I missed that while looking for OID dependencies :-(
Good catch!

            regards, tom lane

Re: Small FK patch to deal with tables without oids

From
Bruce Momjian
Date:
Tom, I assume you want this applied?

---------------------------------------------------------------------------

> Stephan Szabo <sszabo@megazone23.bigpanda.com> writes:
> >     Tables without oids wouldn't be able to be
> > used inside fk constraints, since some of the checks
> > in the trigger did a SELECT oid.  Since the oid wasn't
> > actually used, I changed this to SELECT 1.
>
> Can't believe I missed that while looking for OID dependencies :-(
> Good catch!
>
>             regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
>

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

Re: Small FK patch to deal with tables without oids

From
Tom Lane
Date:
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Tom, I assume you want this applied?

Please.

            regards, tom lane


> ---------------------------------------------------------------------------

>> Stephan Szabo <sszabo@megazone23.bigpanda.com> writes:
> Tables without oids wouldn't be able to be
> used inside fk constraints, since some of the checks
> in the trigger did a SELECT oid.  Since the oid wasn't
> actually used, I changed this to SELECT 1.
>>
>> Can't believe I missed that while looking for OID dependencies :-(
>> Good catch!
>>
>> regards, tom lane
>>
>> ---------------------------(end of broadcast)---------------------------
>> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
>>

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

Re: Small FK patch to deal with tables without oids

From
Bruce Momjian
Date:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > Tom, I assume you want this applied?
>
> Please.

Done.

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

Re: Small FK patch to deal with tables without oids

From
Bruce Momjian
Date:
Patch applied.  Thanks.

---------------------------------------------------------------------------


>
>     Tables without oids wouldn't be able to be
> used inside fk constraints, since some of the checks
> in the trigger did a SELECT oid.  Since the oid wasn't
> actually used, I changed this to SELECT 1.  My test
> case with non-oid tables now works and fk regression
> appears to run fine on my machine.

Content-Description:

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org

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