On Fri, Oct 23, 2009 at 1:41 PM, Jeff <threshar at torgo.978.org> wrote:
> The ENABLE_SUBSCRIPTION event runs in a while(true) loop.
> First it executes query1 (which thanks to the above, is empty), then tries
> to copy_set.  If copy_set fails for whatever reason we ROLLBACK our local
> conn (query2) and then loop.
>

I think this has been a long lasting bug.  When I do a subscription
and the initial copy fails, it *always* issues a warning about commit
outside of a transaction once it succeeds on a subsequent event.  I'll
bet this patch fixes this problem as well.

Nice work!