On Mon, 2004-07-19 at 04:13, Tom Lane wrote:
> Simon Riggs <[EMAIL PROTECTED]> writes:
> > When archiver starts the FIRST thing it does is run a test to confirm
> > that the command string works, so setting archive_command to '' would
> > simply generate an error.
> No, it would do no such thing; the test cannot really tell anything more
> than whether system("foo") returns zero ... and at least on my machine,
> system("") returns zero.  It certainly does not prove that any data went
> to anyplace safe.
> I diked that test out of the committed patch because I felt it cluttered
> the archive area without actually proving anything of interest.  We can
> revisit the point if you like.

If the test doesn't guarantee success, then it needs to go....

Thanks for removing it.

Best Regards, Simon Riggs

---------------------------(end of broadcast)---------------------------
TIP 7: don't forget to increase your free space map settings

Reply via email to