Re: [pgadmin-hackers] Schema bug

2005-12-15 Thread Guillaume Lelarge
2005/12/15, Dave Page : > > Le Mardi 13 Décembre 2005 00:48, Guillaume LELARGE a écrit: > > > Le Samedi 10 Décembre 2005 16:55, Andreas Pflug a écrit: > > > > Dave Page wrote: > > > > >>I still think this patch should be applied. > > > > > > > > > > Does anyone see a reason /not/ to do this? > > >

Re: [pgadmin-hackers] Schema bug

2005-12-14 Thread Guillaume LELARGE
Le Mercredi 14 Décembre 2005 18:14, Andreas Pflug a écrit : > Dave Page wrote: > >>I'm still not convinced we need to do anything. Renaming public is > >>highly irregular, and finally showing system objects will make it > >>reappear. The schema restriction allows individual filters > >>who likes it

Re: [pgadmin-hackers] Schema bug

2005-12-14 Thread Andreas Pflug
Dave Page wrote: I'm still not convinced we need to do anything. Renaming public is highly irregular, and finally showing system objects will make it reappear. The schema restriction allows individual filters who likes it. Renaming public is irregular, but if we can allow it without break

Re: [pgadmin-hackers] Schema bug

2005-12-14 Thread Dave Page
> -Original Message- > From: Andreas Pflug [mailto:[EMAIL PROTECTED] > Sent: 14 December 2005 10:27 > To: Dave Page > Cc: Guillaume LELARGE; pgadmin-hackers@postgresql.org > Subject: Re: [pgadmin-hackers] Schema bug > > > I don't have a problem with tha

Re: [pgadmin-hackers] Schema bug

2005-12-14 Thread Andreas Pflug
Dave Page wrote: -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Guillaume LELARGE Sent: 13 December 2005 23:44 To: pgadmin-hackers@postgresql.org Subject: Re: [pgadmin-hackers] Schema bug Le Mardi 13 Décembre 2005 00:48, Guillaume LELARGE a

Re: [pgadmin-hackers] Schema bug

2005-12-14 Thread Dave Page
> -Original Message- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of > Guillaume LELARGE > Sent: 13 December 2005 23:44 > To: pgadmin-hackers@postgresql.org > Subject: Re: [pgadmin-hackers] Schema bug > > Le Mardi 13 Décembre 2005 00:48

Re: [pgadmin-hackers] Schema bug

2005-12-13 Thread Guillaume LELARGE
Le Mardi 13 Décembre 2005 00:48, Guillaume LELARGE a écrit : > Le Samedi 10 Décembre 2005 16:55, Andreas Pflug a écrit : > > Dave Page wrote: > > >>I still think this patch should be applied. > > > > > > Does anyone see a reason /not/ to do this? > > > > Hm, Guillaume started a thread on pgsql-hack

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Guillaume LELARGE
Le Mardi 13 Décembre 2005 05:58, Andreas Pflug a écrit : > Jim C. Nasby wrote: > > On Tue, Dec 13, 2005 at 12:48:55AM +0100, Guillaume LELARGE wrote: > >>Well, it seems that pg_* schemas are system schemas and that public and > >>information_schema schemas are public one. I think we should exclude

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Andreas Pflug
Jim C. Nasby wrote: On Tue, Dec 13, 2005 at 12:48:55AM +0100, Guillaume LELARGE wrote: Well, it seems that pg_* schemas are system schemas and that public and information_schema schemas are public one. I think we should exclude system schemas depending on their names and show public and inform

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Jim C. Nasby
On Tue, Dec 13, 2005 at 12:48:55AM +0100, Guillaume LELARGE wrote: > Well, it seems that pg_* schemas are system schemas and that public and > information_schema schemas are public one. I think we should exclude system > schemas depending on their names and show public and information_schema, >

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Guillaume LELARGE
Le Samedi 10 Décembre 2005 16:55, Andreas Pflug a écrit : > Dave Page wrote: > >>I still think this patch should be applied. > > > > Does anyone see a reason /not/ to do this? > > Hm, Guillaume started a thread on pgsql-hackers about renaming system > schemas in general, maybe we should wait for th

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Jim C. Nasby
FWIW, here's what we ended up going with in newsysviews: create or replace function _pg_sv_system_schema(name) returns boolean as 'select $1 in (name ''pg_catalog'', name ''pg_toast'', name ''pg_sysviews'', name ''information_schema'')' language sql immutable strict; This

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Florian G. Pflug
Andreas Pflug wrote: > Florian G. Pflug wrote: >> I'd prefer system-catalogs being excluded by name - preferably the >> exclusion-list would be editable, and part of the pgadmin preferences. >> Seems more transparent to me - and future-proof, in the sense that >> even if a future postgres version

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Andreas Pflug
Florian G. Pflug wrote: I'd prefer system-catalogs being excluded by name - preferably the exclusion-list would be editable, and part of the pgadmin preferences. Seems more transparent to me - and future-proof, in the sense that even if a future postgres version chooses to rename some catalog

Re: [pgadmin-hackers] Schema bug

2005-12-12 Thread Florian G. Pflug
Dave Page wrote: I had two ways to fix this bug : detect system schemas on their names, or detect public schema by its OID. I think it's better to only keep the public schema with its OID, it's much less risky to have a wrong one. That's what the patch is doing : AND nspname != 'public' is

Re: [pgadmin-hackers] Schema bug

2005-12-10 Thread Andreas Pflug
Dave Page wrote: I still think this patch should be applied. Does anyone see a reason /not/ to do this? Hm, Guillaume started a thread on pgsql-hackers about renaming system schemas in general, maybe we should wait for the result of that discussion. Regards, Andreas ---

Re: [pgadmin-hackers] Schema bug

2005-12-09 Thread Dave Page
> -Original Message- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of > Guillaume LELARGE > Sent: 09 December 2005 22:43 > To: pgadmin-hackers@postgresql.org > Subject: Re: [pgadmin-hackers] Schema bug > > Le Vendredi 09 Décembre 20

Re: [pgadmin-hackers] Schema bug

2005-12-09 Thread Guillaume LELARGE
Le Vendredi 09 Décembre 2005 09:21, Dave Page a écrit : > > -Original Message- > > From: Florian G. Pflug [mailto:[EMAIL PROTECTED] > > Sent: 09 December 2005 01:15 > > To: Dave Page > > Cc: Andreas Pflug; pgadmin-hackers > > Subject: Re: [pgadmin-hacke

Re: [pgadmin-hackers] Schema bug

2005-12-09 Thread Dave Page
> -Original Message- > From: Florian G. Pflug [mailto:[EMAIL PROTECTED] > Sent: 09 December 2005 01:15 > To: Dave Page > Cc: Andreas Pflug; pgadmin-hackers > Subject: Re: [pgadmin-hackers] Schema bug > > Hm.. couldn't it check the system-schemas by name i

Re: [pgadmin-hackers] Schema bug

2005-12-08 Thread Florian G. Pflug
Dave Page wrote: Andreas Pflug wrote: Guillaume LELARGE wrote: Hi, I found a weird bug today. If you rename the public schema, it becomes unavailable. Here is a patch to fix it. It modifies the query to use the oid instead of the schema's name. Works great on Linux, should'nt be a problem

Re: [pgadmin-hackers] Schema bug

2005-12-08 Thread Andreas Pflug
Florian G. Pflug wrote: Andreas Pflug wrote: Guillaume LELARGE wrote: Hi, I found a weird bug today. If you rename the public schema, it becomes unavailable. Here is a patch to fix it. It modifies the query to use the oid instead of the schema's name. Works great on Linux, should'nt be a

Re: [pgadmin-hackers] Schema bug

2005-12-08 Thread Dave Page
> -Original Message- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of > Florian G. Pflug > Sent: 08 December 2005 15:33 > To: Andreas Pflug; pgadmin-hackers > Subject: Re: [pgadmin-hackers] Schema bug > > Andreas Pflug wrote:

Re: [pgadmin-hackers] Schema bug

2005-12-08 Thread Florian G. Pflug
Andreas Pflug wrote: Guillaume LELARGE wrote: Hi, I found a weird bug today. If you rename the public schema, it becomes unavailable. Here is a patch to fix it. It modifies the query to use the oid instead of the schema's name. Works great on Linux, should'nt be a problem on win32. Actual

Re: [pgadmin-hackers] Schema bug

2005-12-06 Thread Guillaume Lelarge
2005/12/7, Andreas Pflug <[EMAIL PROTECTED]>: > Guillaume LELARGE wrote: > > Hi, > > > > I found a weird bug today. If you rename the public schema, it becomes > > unavailable. Here is a patch to fix it. It modifies the query to use the oid > > instead of the schema's name. Works great on Linux, sh

Re: [pgadmin-hackers] Schema bug

2005-12-06 Thread Andreas Pflug
Guillaume LELARGE wrote: Hi, I found a weird bug today. If you rename the public schema, it becomes unavailable. Here is a patch to fix it. It modifies the query to use the oid instead of the schema's name. Works great on Linux, should'nt be a problem on win32. Actually, to me renaming the

[pgadmin-hackers] Schema bug

2005-12-06 Thread Guillaume LELARGE
Hi, I found a weird bug today. If you rename the public schema, it becomes unavailable. Here is a patch to fix it. It modifies the query to use the oid instead of the schema's name. Works great on Linux, should'nt be a problem on win32. Regards. -- Guillaume. Index: src/schema/pgSchema.cpp