Re: pgsql: Refactor replication connection code of various pg_basebackup ut

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 13:49:31
Message-ID: 20141005134931.GD28859@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

All,

* Andres Freund (andres(at)anarazel(dot)de) wrote:
> Refactor replication connection code of various pg_basebackup utilities.
>
> 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.

Thanks,

Stephen

Attachment Content-Type Size
streamutil-cleanup.patch text/x-diff 2.8 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2014-10-05 14:06:36 Re: pgsql: Refactor replication connection code of various pg_basebackup ut
Previous Message Heikki Linnakangas 2014-10-05 07:06:23 Re: pgsql: Check for GiST index tuples that don't fit on a page.