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-
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 would complicate
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 objecting
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
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_NNN
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
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 test
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 visible
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 can
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
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 bad
11 matches
Mail list logo
