> > so that's what I did, but I think now I have to add a test so only
> > non-visible temp schemas are suppressed,
>
> You are complicating something that could be simple. Why not just
> suppress schemas named 'pg_temp_XXX', period?
Because that's what I originally did and you shot it down as a
Sean Chittenden <[EMAIL PROTECTED]> writes:
> Because that's what I originally did and you shot it down as a bad
> patch because you thought it wasn't in PostgreSQL's interest to filter
> what we showed the user.
I'm still unconvinced on that, actually ... but it beats the heck out of
filtering
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > Also, how do we know something is a temp schema? Just the prefix
> > pg_temp_*?
>
> Yeah. Remember that all schemas named pg_XXX are reserved for system
> use. For the moment, testing for pg_temp_XXX is a bulletproof test,
> and we
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > Tom Lane wrote:
> >> Oh, okay. I would not object to suppressing pg_temp_NNN schemas from
> >> the \dn display. That isn't what this patch does, however.
>
> > OK. I read the TODO and it says only:
> > * Have psql \dn show only
Bruce Momjian <[EMAIL PROTECTED]> writes:
> Also, how do we know something is a temp schema? Just the prefix
> pg_temp_*?
Yeah. Remember that all schemas named pg_XXX are reserved for system
use. For the moment, testing for pg_temp_XXX is a bulletproof test,
and we can certainly adapt psql's te
Bruce Momjian <[EMAIL PROTECTED]> writes:
> Tom Lane wrote:
>> Oh, okay. I would not object to suppressing pg_temp_NNN schemas from
>> the \dn display. That isn't what this patch does, however.
> OK. I read the TODO and it says only:
> * Have psql \dn show only visible schemas using curre
For application to HEAD, pending community review.
Continued rearrangement to permit pgstat + BootstrapMain processes to be
fork/exec'd, in the same mode as the previous patch for backends.
Cheers,
Claudio
---
Certain disclaimers and policies apply to all email sent from Memetrics.
For the ful
Zach Irmen <[EMAIL PROTECTED]> writes:
> A reviewable patch that could handle tilde expansion for
> psql \i ~/filename
Needs a bit of work. What happens if getenv("HOME") returns NULL?
You also need to think about Windows (dunno if the whole thing is
even meaningful there, but if it is then you n
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > Agreed showing just search path and implicit schemas is pretty dumb. I
> > think the issue was that every backend with a temp table was showing up,
> > pretty much swamping the actual schemas he is using.
>
> Oh, okay. I would not o
Bruce Momjian <[EMAIL PROTECTED]> writes:
> Agreed showing just search path and implicit schemas is pretty dumb. I
> think the issue was that every backend with a temp table was showing up,
> pretty much swamping the actual schemas he is using.
Oh, okay. I would not object to suppressing pg_temp
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > This patch uses current_schemas('true') to display only the schemas in
> > the current search path and implicit schemas.
>
> The more I look at this, the sillier it looks. It converts \dn into
> an expensive substitute for "select cu
Tom Lane wrote:
> [EMAIL PROTECTED] (Bruce Momjian) writes:
> > Attached is a patch that addressed all the discussed issues that
> > did not break backward compatability,
>
> Was this patch actually agreed to? I hadn't gotten around to looking
> at it because I thought Peter was still obj
Bruce Momjian <[EMAIL PROTECTED]> writes:
> This patch uses current_schemas('true') to display only the schemas in
> the current search path and implicit schemas.
The more I look at this, the sillier it looks. It converts \dn into
an expensive substitute for "select current_schemas(true)". In
pr
refers to following item on TODO
Have psql '\i ~/' actually load files it displays from home dir
A reviewable patch that could handle tilde expansion for
psql \i ~/filename
or
psql \i ~username/filename
--
Zach Irmen
cvs server: Diffing .
Index: command.c
===
pgman wrote:
> Tom Lane wrote:
> > Bruce Momjian <[EMAIL PROTECTED]> writes:
> > > How about if we add a UNION that does:
> > > UNION
> > > SELECT 'non-local temp schemas skipped', NULL
> >
> > I think showing that would only be appropriate if we actually *did* skip
> > some. Finding that out
Patch applied. Thanks.
I had a little trouble patching shmem.c but got it fixed.
---
Claudio Natoli wrote:
>
> Resubmission, incorporating Neil Conway's comments and some minor
> corrections.
>
> > -Original Message
Patch applied. Thanks.
---
Ron Mayer wrote:
> > -Original Message-
> >
> > Is this ready for application? It looks good to me. However, there is
> > an "Open issues" section.
>
> In my mind there were two categ
17 matches
Mail list logo