Re: improve pg_restore warning on text dump input

Andrew Dunstan <andrew@dunslane.net>

From: Andrew Dunstan <andrew@dunslane.net>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2012-01-03T19:09:28Z
Lists: pgsql-hackers

On 01/03/2012 01:55 PM, Tom Lane wrote:
> Andrew Dunstan<andrew@dunslane.net>  writes:
>>    From time to time there are complaints because people mistakenly feed a
>> text format dump to pg_restore and get back a somewhat cryptic message
>> about the file not being a valid archive. It's been suggested that we
>> should have pg_restore run the file through psql, but that would involve
>> more work than I at least care to give the problem. However, I think we
>> should give a nicer message, suggesting the user try feeding the file to
>> psql instead. The attached small patch does that.
> It would probably be better if you put this test before the one that
> insists the file is at least 512 bytes.
>
> 			


Hmm, yeah. I guess we're pretty much certain that these markers can't 
reasonably appear at the start of a tar archive.

cheers

andrew