On 2014-10-04 14:25:27 +0900, Michael Paquier wrote:
> And as I am on it, attached is a patch that can be applied to master and
> REL9_4_STABLE to rename the --create and --drop to --create-slot and
> --drop-slot.
Thanks.
> diff --git a/src/bin/pg_basebackup/pg_recvlogical.c
> b/src/bin/pg_basebackup/pg_recvlogical.c
> index c48cecc..585d7b0 100644
> --- a/src/bin/pg_basebackup/pg_recvlogical.c
> +++ b/src/bin/pg_basebackup/pg_recvlogical.c
> @@ -91,8 +91,8 @@ usage(void)
> " time between status packets
> sent to server (default: %d)\n"), (standby_message_timeout / 1000));
> printf(_(" -S, --slot=SLOT name of the logical replication
> slot\n"));
> printf(_("\nAction to be performed:\n"));
> - printf(_(" --create create a new replication slot (for
> the slot's name see --slot)\n"));
> - printf(_(" --start start streaming in a replication
> slot (for the slot's name see --slot)\n"));
> + printf(_(" --create-slot create a new replication slot (for
> the slot's name see --slot)\n"));
> + printf(_(" --start-slot start streaming in a replication
> slot (for the slot's name see --slot)\n"));
> printf(_(" --drop drop the replication slot (for the
> slot's name see --slot)\n"));
> printf(_("\nReport bugs to <[email protected]>.\n"));
> }
> @@ -618,9 +618,9 @@ main(int argc, char **argv)
> {"status-interval", required_argument, NULL, 's'},
> {"slot", required_argument, NULL, 'S'},
> /* action */
> - {"create", no_argument, NULL, 1},
> + {"create-slot", no_argument, NULL, 1},
> {"start", no_argument, NULL, 2},
> - {"drop", no_argument, NULL, 3},
> + {"drop-slot", no_argument, NULL, 3},
> {NULL, 0, NULL, 0}
Uh? You're documenting --start-slot here and a couple other places, but
you implement --drop-slot. And --start-slot doesn't seem to make much
sense to me.
I've pushed a polished up version. Fixing the above and
logicaldecoding.sgml
Note that at least gnu's getopt_long() accepts --create for
--create-slot...
Thanks,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
--
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
