Re: pg_receivexlog and replication slots

On Fri, Oct 3, 2014 at 8:57 PM, Andres Freund <andres@2ndquadrant.com> wrote:
> >     <para>
> > +    <application>pg_receivexlog</application> can run in one of two following
> > +    modes, which control physical replication slot:
>
> I don't think that's good enough. There's also the important mode where
> it's not doing --create/--drop at all.
Well, yes, however the third mode is not explicitly present, and I
don't see much point in adding a --start mode thinking
backward-compatibility. Now, I refactored a bit the documentation to
mention that pg_receivexlog can perform additional actions to control
replication slots. I added as well in the portion of option --slot how
it interacts with --create-slot and --drop-slot.

> > +     if (db_name)
> > +     {
> > +             fprintf(stderr,
> > +                             _("%s: database defined for replication connection \"%s\"\n"),
> > +                             progname, replication_slot);
> > +             disconnect_and_exit(1);
> > +     }
>
> I don't like 'defined' here. 'replication connection unexpectedly is
> database specific' or something would be better.

Sure, IMO the error message should as well mention the replication
slot being used, so I reformulated as such:
"replication connection using slot foo is unexpectedly database specific"

>
> I do wonder whether --create/--drop aren't somewhat weird for
> pg_receivexlog. It's not that clear what it means. It'd be ugly, but we
> could rename them --create-slot/drop-slot.

In line with the other patch sent earlier, options are renamed to
--create-slot and --drop-slot.
Regards,
--
Michael