Tom Lane wrote:
> Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> > Tom Lane wrote:
> >> Yeah, an extra fetch of the pg_class row doesn't seem all that nice.
> >> I think you'd want to check it in approximately the same two places
> >> where pg_class_ownercheck() is applied (one for the 1-xact and one for
> >> the multi-xact path).
> > Actually, the 1-xact path does not need it, because the check is already
> > elsewhere.
> Yeah, but if you do it there it's one added comparison
> (isOtherTempNamespace is very cheap, and you can get the namespace
> cheaply from the already-open rel). This way you need an extra syscache
> lookup because you are insisting on doing the check in a place where you
> don't have easy access to the pg_class row. Doesn't seem better.
I'm not sure I follow. Are you suggesting adding a new function,
similar to pg_class_ownercheck, which additionally checks for temp-ness?
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
In response to
pgsql-hackers by date
|Next:||From: Oleg Bartunov||Date: 2007-08-30 17:03:00|
|Subject: Re: Why is there a tsquery data type?|
|Previous:||From: Charlie Savage||Date: 2007-08-30 16:41:13|
|Subject: Re: Some more msvc++ 8.2.4 build feedback|
pgsql-admin by date
|Next:||From: Tom Lane||Date: 2007-08-30 18:54:21|
|Subject: Re: reindexdb hangs |
|Previous:||From: Jessica Richard||Date: 2007-08-30 16:09:51|
|Subject: Postgres can be started only when one postgres user is logged in|