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

While I see both sides, this looks like an *INTERNAL* *CORE* debate.

From a USER perspective the functionality is useful. From a Software
Development perspective, the timing stinks.

I'd leave it in /contrib, and make damn sure we get the right
functionality into the 7.1 release.

LER

* Vince Vielhaber <vev(at)michvhf(dot)com> [001102 19:18]:
> On Thu, 2 Nov 2000, The Hermit Hacker wrote:
>
> > On Thu, 2 Nov 2000, Bruce Momjian wrote:
> >
> > > > Ned Lilly <ned(at)greatbridge(dot)com> writes:
> > > > > Well, here in relatively minor form is the First Example of a Great
> > > > > Bridge Priority (which Tom, Bruce, and Jan have all predicted would
> > > > > come... ;-)
> > > >
> > > > Hmm. I wasn't aware that Jan had done it at Great Bridge's request,
> > > > and I am going to make a point of not letting that affect my opinion ;-).
> > > >
> > > > What really got my ire up was that this change was committed several
> > > > days *after* core had agreed that 7.0.3 was frozen and ready to go except
> > > > for updating the changelog, and that it was committed with no prior
> > > > notice or discussion. The fact that GB asked for it doesn't make that
> > > > better; if anything it makes it worse. We wouldn't have accepted such
> > > > a patch at this late date from an outside contributor, I believe.
> > > > Jan should surely have known better than to handle it in this fashion.
> > > >
> > > > Need I remind you, also, that GB has been bugging us for several weeks
> > > > to get 7.0.3 released ASAP? Last-minute changes don't further that
> > > > goal.
> > > >
> > > > The early returns from pghackers seem to be that people favor just
> > > > dropping the script into /contrib and not worrying about how well
> > > > tested/documented it is. If that's the consensus then I'll shut up
> > > > ... but I do *not* like the way this was handled.
> > >
> > > I totally agree with Tom on all his points. If people were worried we
> > > would not be objective now that we are employed by GB, they can rest
> > > easy.
> > >
> > > Also, seems like it is hidden enough in /contrib for it to stay. While
> > > I would not have added it myself, I do not feel strongly enough to
> > > remove Jan's commit. However, I am not going to mention it in the 7.0.3
> > > release notes.
> >
> > I do feel strongly about this ... 7.0.3 was considered in a release state
> > *before* it was committed, pending your docs changes ... personally, if we
> > leave this in contrib, my vote is to hold off the release a suitable
> > amount of time for testing purposes ... Jan has added a new feature that
> > nobody had any pre-warning about, not even other developers in the same
> > company as he is in ... not a good precedent :(
>
> What am I missing? We're talking about contrib. Most things you find
> in contrib directories don't even work and you're worried about a testing
> phase? Most folks don't even look in contrib directories unless they're
> specifically looking for something.
>
> Vince.
> --
> ==========================================================================
> Vince Vielhaber -- KA8CSH email: vev(at)michvhf(dot)com http://www.pop4.net
> 128K ISDN from $22.00/mo - 56K Dialup from $16.00/mo at Pop4 Networking
> Online Campground Directory http://www.camping-usa.com
> Online Giftshop Superstore http://www.cloudninegifts.com
> ==========================================================================
>
>

--
Larry Rosenman http://www.lerctr.org/~ler
Phone: +1 972-414-9812 (voice) Internet: ler(at)lerctr(dot)org
US Mail: 1905 Steamboat Springs Drive, Garland, TX 75044-6749