Re: Re: [COMMITTERS] pgsql/contrib/pg_dumpaccounts (Makefile README pg_dumpaccounts.sh)

From: The Hermit Hacker <scrappy(at)hub(dot)org>
To: Ned Lilly <ned(at)greatbridge(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Jan Wieck <janwieck(at)Yahoo(dot)com>, PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [COMMITTERS] pgsql/contrib/pg_dumpaccounts (Makefile README pg_dumpaccounts.sh)
Date: 2000-11-03 00:29:52
Message-ID: Pine.BSF.4.21.0011022025120.494-100000@thelab.hub.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, 2 Nov 2000, Ned Lilly wrote:

> We recognize this is a temporary hack - and fully expect it to go away
> in 7.1 We actually think that the final solution might be more
> appropriate in pg_dump itself than pg_dumpall, but that's obviously a
> much more breakable proposition (hence the separate utility).

Okay, because of this paragraph, and this one only, I will agree with Tom
(and I believe Bruce) that this should be removed. If, as Peter states,
this could be put into pg_dump in 5 lines, and as you say, it is a
temporary hack, then more appropriate would be to put a link off of the
web site and *not* put it into the source distribution ...

I like what it does, since I can relate to the need to dump user/group
info seperate from everything else, but if a permanent fix is as doable as
Peter states, putting a temporary one, especially into a minor release,
makes little to no sense ...

My vote is to please remove it from the source tree ...

> If it *is* necessary (or at least worthwhile), is it the consensus of
> the -hackers community that it go in /contrib?

Altho this is going to force me to agree with Tom concerning Karel's
patch, it should not be added to the 7.0.x branch *at all* ... 7.0.x is a
*patch* release, new features are for 7.1 and 7.1 only ...

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message The Hermit Hacker 2000-11-03 00:34:56 Re: Re: [COMMITTERS] pgsql/contrib/pg_dumpaccounts (Makefile README pg_dumpaccounts.sh)
Previous Message tgl 2000-11-02 23:52:07 pgsql/src/backend/storage/large_object (inv_api.c)

Browse pgsql-hackers by date

  From Date Subject
Next Message The Hermit Hacker 2000-11-03 00:34:56 Re: Re: [COMMITTERS] pgsql/contrib/pg_dumpaccounts (Makefile README pg_dumpaccounts.sh)
Previous Message Bruce Momjian 2000-11-03 00:28:16 Re: Re: [CORE] 7.0.3 Release date?