On Tue, 2006-02-14 at 09:35 -0500, Tom Lane wrote:
> Simon Riggs <simon@2ndquadrant.com> writes:
> > On Tue, 2006-02-14 at 16:55 +0800, Christopher Kings-Lynne wrote:
> >> Huh? But it'll cause a total failure of dump restore?
>
> > Perhaps you can explain further?
>
> As the code stands, a restore involving blobs plus --single-transaction
> produces
> BEGIN;
> ...
> BEGIN;
> ...
> COMMIT;
> ...
> COMMIT;
> which does *not* have the intended behavior because BEGIN does not nest.
> This is a must-fix, else we may as well revert the patch entirely,
> because it does not work.
How should it work?
1. Remove the BEGIN and COMMIT around blobs?
2. Use SAVEPOINT ?
Presumably (1).
Best Regards, Simon Riggs