Re: RE: [PATCHES] relation filename patch

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: "Ross J(dot) Reedstrom" <reedstrm(at)wallace(dot)ece(dot)rice(dot)edu>, Hiroshi Inoue <Inoue(at)tpf(dot)co(dot)jp>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, pgsql-patches(at)postgresql(dot)org
Subject: Re: RE: [PATCHES] relation filename patch
Date: 2000-05-02 21:28:14
Message-ID: Pine.LNX.4.21.0005020034140.389-100000@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Bruce Momjian writes:

> > Hmm. Further perusal leads me to believe that this override is a
> > standards extension,

> I am quite surprised people don't like that feature, or at least one
> person doesn't.

FWIW, I always considered that behaviour kind of suspicious. I don't think
it's a big problem in practice and could be useful in certain situations,
but personally I would have chosen not to do it this way.

> If someone else creates a table, it should not prevent me from
> creating a temporary table of the same name.

The proper solution to this is that "somebody" should not be allowed to
create tables wildly. Until we can properly do that it's not worth arguing
this out.

> Our code even masks a real table created in the same session. Once
> the temp table is dropped, the real table becomes visible again. See
> the regression tests for an example of this.

The real problem here is that there's no way of finding out whether you
just dropped the temporary table or the "real" one or whether a table is
temporary at all. Sure you can perhaps look into pg_class but tell that to
users.

--
Peter Eisentraut Sernanders väg 10:115
peter_e(at)gmx(dot)net 75262 Uppsala
http://yi.org/peter-e/ Sweden

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2000-05-02 21:29:06 Re: Proposal for fixing intra-query memory leaks
Previous Message Bruce Momjian 2000-05-02 21:26:48 Re: RE: [PATCHES] relation filename patch