Re: Remove pg_dump -i option (was Re: Proposed patch: synchronized_scanning GUC variable)

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Neil Conway <neilc(at)samurai(dot)com>, Gregory Stark <stark(at)enterprisedb(dot)com>, "Jonah H(dot) Harris" <jonah(dot)harris(at)gmail(dot)com>
Subject: Re: Remove pg_dump -i option (was Re: Proposed patch: synchronized_scanning GUC variable)
Date: 2008-01-31 15:14:58
Message-ID: 200801311514.m0VFEwY24520@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Peter Eisentraut wrote:
> Am Donnerstag, 31. Januar 2008 schrieb Alvaro Herrera:
> > Effect: we would stop receiving complaints that an old pg_dump can talk
> > to a server that most likely is incompatible with it. People would
> > learn on the spot that they must install the newer pg_dump.
>
> I think a more moderate measure might be to clarify the error message
>
> "aborting because of version mismatch (Use the -i option to proceed
> anyway.)\n"
>
> I'm not sure how many of the mentioned complaints we are getting, but the
> error message might make people think that it's complaining because the
> versions are not equal rather than that the versions are known not
> compatible.

Agreed --- that error message is just prompting people to try something
they shouldn't and not explaining why.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2008-01-31 15:21:09 Re: Oops - BF:Mastodon just died
Previous Message Peter Eisentraut 2008-01-31 15:13:55 Re: Remove pg_dump -i option (was Re: Proposed patch: synchronized_scanning GUC variable)

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2008-01-31 15:25:54 Re: {**Spam**} Re: [PATCHES] Proposed patch: synchronized_scanning GUC variable
Previous Message Peter Eisentraut 2008-01-31 15:13:55 Re: Remove pg_dump -i option (was Re: Proposed patch: synchronized_scanning GUC variable)