Re: [HACKERS] adminpack and pg_catalog

2006-11-13 Thread Dave Page
Robert Treat wrote: While I don't disagree that this is an important feature, the fact that it is being designed with pgadmin specific backwards compatability (for example the functions that rename core functions) leaves me dubious as to it being a more general solution. Because of that I

Re: [HACKERS] adminpack and pg_catalog

2006-11-12 Thread Robert Treat
On Monday 06 November 2006 13:12, Simon Riggs wrote: On Mon, 2006-11-06 at 09:02 +, Dave Page wrote: Neil Conway wrote: On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special

Re: [HACKERS] adminpack and pg_catalog

2006-11-06 Thread Dave Page
Neil Conway wrote: On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special requirements, so it should behave like all other contrib modules. Where are we on this? When this topic was last

Re: [HACKERS] adminpack and pg_catalog

2006-11-06 Thread Simon Riggs
On Mon, 2006-11-06 at 09:02 +, Dave Page wrote: Neil Conway wrote: On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special requirements, so it should behave like all other contrib

Re: [HACKERS] adminpack and pg_catalog

2006-11-06 Thread Tom Lane
Simon Riggs [EMAIL PROTECTED] writes: At the moment we only allow 2 types of table. Approved core catalog tables and user tables. ISTM we need 3 types of tables, with the additional type being add-on system functionality, such as adminpack, What? The adminpack module only creates functions.

Re: [HACKERS] adminpack and pg_catalog

2006-11-06 Thread Simon Riggs
On Mon, 2006-11-06 at 13:37 -0500, Tom Lane wrote: Simon Riggs [EMAIL PROTECTED] writes: At the moment we only allow 2 types of table. Approved core catalog tables and user tables. ISTM we need 3 types of tables, with the additional type being add-on system functionality, such as

Re: [HACKERS] adminpack and pg_catalog

2006-11-05 Thread Neil Conway
On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special requirements, so it should behave like all other contrib modules. Where are we on this? When this topic was last discussed, the three

Re: [HACKERS] adminpack and pg_catalog

2006-10-22 Thread Simon Riggs
On Sat, 2006-10-21 at 12:37 +0200, Peter Eisentraut wrote: Dave Page wrote: If you change it you will make it useless as pgAdmin won't necessarily find the functions it expects. You might as well just remove it (which will almost certainly cause delays to pgAdmin - and pgInstallers -

Re: [HACKERS] adminpack and pg_catalog

2006-10-21 Thread Dave Page
-Original Message- From: Neil Conway [EMAIL PROTECTED] To: Peter Eisentraut [EMAIL PROTECTED] Cc: pgsql-hackers@postgresql.org pgsql-hackers@postgresql.org; Dave Page dpage@vale-housing.co.uk Sent: 21/10/06 02:03 Subject: Re: [HACKERS] adminpack and pg_catalog On Fri, 2006-10-20 at 22

Re: [HACKERS] adminpack and pg_catalog

2006-10-21 Thread Dave Page
-Original Message- From: Neil Conway [EMAIL PROTECTED] To: Dave Page dpage@vale-housing.co.uk Cc: PostgreSQL-development pgsql-hackers@postgresql.org Sent: 20/10/06 21:19 Subject: Re: adminpack and pg_catalog It breaks in the sense of completely not working :) No, it does not 'break

Re: [HACKERS] adminpack and pg_catalog

2006-10-21 Thread Peter Eisentraut
Dave Page wrote: If you change it you will make it useless as pgAdmin won't necessarily find the functions it expects. You might as well just remove it (which will almost certainly cause delays to pgAdmin - and pgInstallers - release as I'll need to find time to put it all back how it was).

Re: [HACKERS] adminpack and pg_catalog

2006-10-21 Thread Robert Treat
On Friday 20 October 2006 21:03, Neil Conway wrote: On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special requirements, so it should behave like all other contrib modules. Okay. Are

FW: [pgadmin-hackers] FW: [HACKERS] adminpack and pg_catalog

2006-10-21 Thread Dave Page
-Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Dave Page Sent: 21 October 2006 21:20 To: PgAdmin Hackers Subject: [pgadmin-hackers] FW: [HACKERS] adminpack and pg_catalog [Ooops, forgot to CC the list] And then got the wrong one. D'oh

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Andreas Pflug
Neil Conway wrote: Why does adminpack install functions into pg_catalog? This is inconsistent with the rest of the contrib/ packages, not to mention the definition of pg_catalog itself (which ought to hold builtin object definitions). And as AndrewSN pointed out on IRC, it also breaks

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Neil Conway
On Fri, 2006-10-20 at 11:50 +0200, Andreas Pflug wrote: Having pg_dump not saving the function definitions is an intended behaviour. The manual defines the pg_catalog schema as containing the system tables and all the built-in data types, functions, and operators (section 5.7.5). adminpack is

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Neil Conway
On Fri, 2006-10-20 at 05:52 +0100, Dave Page wrote: The adminpack was originally written and intended to become builtin functions This is not unique to adminpack: several contrib modules might eventually become (or have already become) builtins, but adminpack is the only module that defines

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Peter Eisentraut
Neil Conway wrote: Why does adminpack install functions into pg_catalog? This is inconsistent with the rest of the contrib/ packages, not to mention the definition of pg_catalog itself (which ought to hold builtin object definitions). Nothing except initdb should add objects in pg_catalog.

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Andreas Pflug
Neil Conway wrote: On Fri, 2006-10-20 at 05:52 +0100, Dave Page wrote: The adminpack was originally written and intended to become builtin functions This is not unique to adminpack: several contrib modules might eventually become (or have already become) builtins, but adminpack is

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Neil Conway
On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special requirements, so it should behave like all other contrib modules. Okay. Are there any opinions on whether we should make this change

Re: [HACKERS] adminpack and pg_catalog

2006-10-20 Thread Tom Lane
Neil Conway [EMAIL PROTECTED] writes: On Fri, 2006-10-20 at 22:59 +0200, Peter Eisentraut wrote: Nothing except initdb should add objects in pg_catalog. AFAICS, adminpack doesn't have any special requirements, so it should behave like all other contrib modules. Okay. Are there any

[HACKERS] adminpack and pg_catalog

2006-10-19 Thread Neil Conway
Why does adminpack install functions into pg_catalog? This is inconsistent with the rest of the contrib/ packages, not to mention the definition of pg_catalog itself (which ought to hold builtin object definitions). And as AndrewSN pointed out on IRC, it also breaks pg_dump. -Neil

Re: [HACKERS] adminpack and pg_catalog

2006-10-19 Thread Jim C. Nasby
On Thu, Oct 19, 2006 at 02:37:34PM -0400, Neil Conway wrote: Why does adminpack install functions into pg_catalog? This is inconsistent with the rest of the contrib/ packages, not to mention the definition of pg_catalog itself (which ought to hold builtin object definitions). And as AndrewSN

Re: [HACKERS] adminpack and pg_catalog

2006-10-19 Thread Dave Page
On 19/10/06 19:37, Neil Conway [EMAIL PROTECTED] wrote: Why does adminpack install functions into pg_catalog? This is inconsistent with the rest of the contrib/ packages, not to mention the definition of pg_catalog itself (which ought to hold builtin object definitions). The adminpack