From: Tom Lane <tgl@sss.pgh.pa.us>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Andrew Dunstan <andrew@dunslane.net>
Cc: Stephen Frost <sfrost@snowman.net>
Cc: Joshua D. Drake <jd@commandprompt.com>
Cc: pgsql-hackers@postgresql.org <pgsql-hackers@postgresql.org>
Subject: Re: RFC: Remove contrib entirely
Date: Thu, 04 Jun 2015 11:57:36 -0400
Message-ID: <11341.1433433456@sss.pgh.pa.us> (raw)
In-Reply-To: <CA+TgmoZ2xMM1JB9YnuOQXQ-GOqaV-qTeC5_unSeUkahtnz3=Aw@mail.gmail.com>
References: <55674016.80005@commandprompt.com>
<20150528193556.GH26667@tamriel.snowman.net>
<5567C5EB.3070703@gmx.net>
<5567CB9E.1030502@commandprompt.com>
<20150529031004.GU26667@tamriel.snowman.net>
<5567DDBA.8020403@commandprompt.com>
<CAECtzeWrOT7LgNABk9V=CWCATHYQMxLdE9etG3iHGADqa8kOQA@mail.gmail.com>
<alpine.DEB.2.10.1505290750050.24520@sto>
<55686D84.4040202@commandprompt.com>
<20150529143026.GV26667@tamriel.snowman.net>
<CA+Tgmoa+hEZUmXCp+KqsZdMRdRVM36KgWNqd-momdmKVd-UU9Q@mail.gmail.com>
<55706D1D.4040606@dunslane.net>
<CA+TgmoZ2xMM1JB9YnuOQXQ-GOqaV-qTeC5_unSeUkahtnz3=Aw@mail.gmail.com>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
Robert Haas <robertmhaas@gmail.com> writes:
> On Thu, Jun 4, 2015 at 11:22 AM, Andrew Dunstan <andrew@dunslane.net> wrote:
>> The biggest problem is that packagers tend just to bundle contrib together
>> in one lump. If we could divide it into two, something like "standard
>> modules" and "misc", with the former being included with the server package,
>> I think that would be an advance, although packagers might reasonably want
>> to treat pgcrypto as a special case.
As an ex-packager, I agree that would be a good thing. In particular
we should try to avoid people packaging stuff that's meant either for
server testing or as sample-source-code-not-useful-in-itself. We've
made some progress on the former via src/test/modules but I wonder
if we're all the way there; test_decoding surely shouldn't be in
contrib on this measure should it?
BTW, pg_upgrade is also a special case from a packager's viewpoint,
since it needs to be bundled with back-version executables.
> The problem is that it's very hard to agree on which stuff ought to be
> standard and which stuff ought to be misc. There's probably some
> stuff that almost everyone would agree is pretty useful (like hstore
> and postgres_fdw) but figuring out which stuff isn't useful is a lot
> harder. Almost anything you say - that's junk - someone else will say
> - no, that thing is great, I use it all the time. For example, I just
> offered contrib/isn as a sort of archetypal example of stuff that's
> pretty marginal crap and Josh immediately came back and said, hey, I
> use that! I don't see any principled way of getting past that
> difficulty. Just because a module isn't regularly used by someone who
> reads -hackers daily doesn't mean it's not worth keeping.
Yeah. One possible way of measuring this would be to go through the
commit logs and count commits in contrib/ that either added a new
feature or fixed a field-reported bug (ie, did not arise simply from
core-code-driven housekeeping). Either would be solid evidence that
somebody out there is using it. Gathering the evidence would be
pretty tedious though :-(
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, robertmhaas@gmail.com, andrew@dunslane.net, sfrost@snowman.net, jd@commandprompt.com
Subject: Re: RFC: Remove contrib entirely
In-Reply-To: <11341.1433433456@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox