On Sun, Jul 25, 2010 at 2:37 AM, Jaime Casanova <jaime(at)2ndquadrant(dot)com> wrote:
> but i have a few questions, maybe is right what you did i only want to
> understand it:
> - you added this in include/storage/smgr.h, so why is safe to assume
> that if the backend != InvalidBackendId it must be a temp relation?
> +#define SmgrIsTemp(smgr) \
> + ((smgr)->smgr_rnode.backend != InvalidBackendId)
That's pretty much the whole point of the patch. Instead of
identifying relations as simply "temporary" or "not temporary", we
identify as "a temporary relation owned by backend X" or as "not
> - you added a question like this "if (rel->rd_backend == MyBackendId)"
> in a few places... why are you assuming that? that couldn't be a new
> created relation (in current session of course)? is that safe?
Again, rd_backend is not the creating backend ID unless the relation
is a temprel.
The Enterprise Postgres Company
In response to
pgsql-hackers by date
|Next:||From: Robert Haas||Date: 2010-07-25 21:32:57|
|Subject: Re: non-overlapping, consecutive partitions|
|Previous:||From: Pavel Stehule||Date: 2010-07-25 15:42:57|
|Subject: Re: review: psql: edit function, show function commands patch|