From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Refactor replication connection code of various pg_basebackup ut |
Date: | 2014-10-05 14:11:11 |
Message-ID: | 20141005141111.GE28859@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
* Andres Freund (andres(at)anarazel(dot)de) wrote:
> On 2014-10-05 09:49:31 -0400, Stephen Frost wrote:
> > > Move some more code to manage replication connection command to
> > > streamutil.c. A later patch will introduce replication slot via
> > > pg_receivexlog and this avoid duplicating relevant code between
> > > pg_receivexlog and pg_recvlogical.
> >
> > Coverity is complaining that the new Create/DropReplicatSlot functions
> > leak, which is technically correct (though the cases where that happens
> > end up being more-or-les immediate-exit cases anyway). In any case,
> > here's a patch to clean up things prior to return.
>
> I can't get very excited about this - but I also have no problems fixing
> it.
Ok, great.
> I'm fine with you committing what you have. Or would you rather have
> me do it?
I'll do it in a few hours. I'm playing w/ pg_recvlogical. ;)
Thanks!
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2014-10-05 14:40:40 | Re: Re: pgsql: Refactor replication connection code of various pg_basebackup ut |
Previous Message | Andres Freund | 2014-10-05 14:06:36 | Re: pgsql: Refactor replication connection code of various pg_basebackup ut |