Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader - Mailing list pgsql-hackers

From Peter Geoghegan
Subject Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader
Date
Msg-id CAEYLb_XJ4NMKZ0YvN2F9w3L1qbg74NX3XcGOLRt57bYf8Npqiw@mail.gmail.com
Whole thread Raw
In response to Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader  (Dimitri Fontaine <dimitri@2ndQuadrant.fr>)
Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader  (Andres Freund <andres@2ndquadrant.com>)
List pgsql-hackers
On 11 December 2012 22:24, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> TBH, I don't believe that ordinary users will need this tool at all,
> ever, and thus I don't want it in src/bin/.  From a packaging standpoint
> it will be a lot easier if it's in contrib ... otherwise I'll probably
> have to invent some new sub-RPM along the lines of postgresql-extras
> so as to avoid bloating the core server package.

I happen to agree that pg_xlogdump belongs in contrib, but I think
that the importance of avoiding "bloat" has been overstated. Maybe
it's slightly useful to make sure that Postgres can get on the Fedora
CD, but that aside, is including pg_xlogdump here, for example, really
likely to make any appreciable difference package-wise?

pg_xlogdump is 141K on my system. I'd hate to see us embrace the exact
opposite tendency, towards including everything but the kitchen sink,
but at the same time that seems like a very insignificant size.
Perhaps people who live in countries with less bandwidth care about
these things more.

-- 
Peter Geoghegan       http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training and Services



pgsql-hackers by date:

Previous
From: Dimitri Fontaine
Date:
Subject: Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader
Next
From: Dimitri Fontaine
Date:
Subject: Re: Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader