On 08/26/2016 07:03 PM, David Fetter wrote:
On Fri, Aug 26, 2016 at 01:26:39PM -0300, Euler Taveira wrote:
Hi,

...
>>
initdb: we already have 'pg_ctl init' (since 9.0) and could remove initdb.

Opinions?

+1 for removing initdb.


We can't just remove it because pg_ctl actually calls initdb, so we would have to move the code from src/bin/initdb to src/bin/pg_ctl. Which is doable, but I assume we'd also change how we pass parameters to it - right now we have to do this:

pg_ctl init -o "... initdb params ..."

But I assume we'd change this to regular parameters, if 'init' gets merged into pg_ctl - otherwise we'd force users to construct the param string only to immediately parse it (currently the "system()" call does that for us, but we'd have to implement that in pg_ctl).

That however means that not only we break the scripts for all those calling initdb directly (and there's quite a few of them, I'm sure, exactly because it's simpler to pass the parameters directly), but also for all those already using pg_ctl.

regards

--
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to