Skip site navigation (1) Skip section navigation (2)

Re: pg_restore ignore error patch

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
Cc: PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: pg_restore ignore error patch
Date: 2004-04-20 12:58:26
Message-ID: 200404201258.i3KCwQp29293@candle.pha.pa.us (view raw or flat)
Thread:
Lists: pgsql-patches
Fabien COELHO wrote:
> 
> > > > I looked over the patch and it seems to continue on pg_restore errors by
> > > > default.  That isn't good.  By default, any error should make it exit
> > > > loudly.
> > >
> > > I'm not sure of that.  pg_dump is really designed and tested for the
> > > case of text dump to a psql script, and if there is an error in the psql
> > > [...]
> >
> > Oh, OK, so make it behave like pg_dump's text output piped into psql.
> 
> It is really easy to add an option to allow user change the 'ignore'
> behavior, and make pg_restore exit loudly if it is desired.
> 
> Maybe it should be proposed just for backwards compatibility?

Let's see if anyone asks for it.  Unless they do, we will just enable it
by default.  As long as we exit with a non-zero status on error, just
like psql, I think we are OK.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

In response to

pgsql-patches by date

Next:From: Bruce MomjianDate: 2004-04-20 13:22:00
Subject: COPY CSV keywords
Previous:From: Tom LaneDate: 2004-04-20 12:10:43
Subject: Re: Basic subtransaction facility

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group