Re: detect custom-format dumps in psql and emit a useful error - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: detect custom-format dumps in psql and emit a useful error
Date
Msg-id 20141024101855.GL1791@alvin.alvh.no-ip.org
Whole thread Raw
In response to Re: detect custom-format dumps in psql and emit a useful error  (Jeevan Chalke <jeevan.chalke@enterprisedb.com>)
Responses Re: detect custom-format dumps in psql and emit a useful error
List pgsql-hackers
Jeevan Chalke wrote:

> Regarding Directory Error:
> ===
> I strongly against the proposal. This patch changing error message to
> something like this:
> "psql:blah:0: Input path is a directory. Use pg_restore to restore
> directory-format database dumps."
> 
> So even though I accidentally provide a directory instead of a sql script
> file when I have NO intention of restoring a dump, above message looks
> weired. Instead current message looks perfectly fine here. i.e.
> "could not read from input file: Is a directory"
> 
> psql always expect a file and NOT directory. Also it is not necessarily
> working on restoring a dump.

Yeah, this patch is a lot more debatable than the other one.  I have
pushed the first one without changing the error message.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services



pgsql-hackers by date:

Previous
From: Craig Ringer
Date:
Subject: TODO: Helpful hint from psql on createdb
Next
From: Andres Freund
Date:
Subject: Re: detect custom-format dumps in psql and emit a useful error