Re: pg_restore COPY error handling

Stephen Frost wrote:
-- Start of PGP signed section.
> Bruce, et al,
>
> * Bruce Momjian (pgman@candle.pha.pa.us) wrote:
> > This needs a little style cleanup but I can do that.
>
> Thanks!
>
> > Your patch has been added to the PostgreSQL unapplied patches list at:
> >
> >     http://momjian.postgresql.org/cgi-bin/pgpatches
> >
> > It will be applied as soon as one of the PostgreSQL committers reviews
> > and approves it.
>
> Great!  It'd be really nice to have this fix in 8.1.3... :)

No, it will not be in 8.1.3.  It is a new feature.

---------------------------------------------------------------------------


>
>     Thanks again,
>
>         Stephen
>
> > ---------------------------------------------------------------------------
> > Stephen Frost wrote:
> > > * Stephen Frost (sfrost@snowman.net) wrote:
> > > > Needs to be changed to handle whitespace in front of the actual 'COPY',
> > > > unless someone else has a better idea.  This should be reasonably
> > > > trivial to do though...  If you'd like me to make that change and send
> > > > in a new patch, just let me know.
> > >
> > > Fixed, using isspace().  Also added an output message to make it a bit
> > > clearer when a failed COPY has been detected, etc.
> > >
> > > Updated patch attached.
> > >
> > >     Thanks,
> > >
> > >         Stephen
> >
> > [ Attachment, skipping... ]
> >
> > >
> > > ---------------------------(end of broadcast)---------------------------
> > > TIP 4: Have you searched our list archives?
> > >
> > >                http://archives.postgresql.org
> >
> > --
> >   Bruce Momjian                        |  http://candle.pha.pa.us
> >   pgman@candle.pha.pa.us               |  (610) 359-1001
> >   +  If your life is a hard drive,     |  13 Roberts Road
> >   +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
-- End of PGP section, PGP failed!

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