Rename pg_recvlogical's --create/--drop to --create-slot/--drop-slot.

A future patch (9.5 only) adds slot management to pg_receivexlog. The
verbs create/drop don't seem descriptive enough there. It seems better
to rename pg_recvlogical's commands now, in beta, than live with the
inconsistency forever.

The old form (e.g. --drop) will still be accepted by virtue of most
getopt_long() options accepting abbreviations for long commands.

Backpatch to 9.4 where pg_recvlogical was introduced.

Author: Michael Paquier and Andres Freund
Discussion: CAB7nPqQtt79U6FmhwvgqJmNyWcVCbbV-nS72j_jyPEopERg9rg@mail.gmail.com

diff --git a/doc/src/sgml/logicaldecoding.sgml b/doc/src/sgml/logicaldecoding.sgml
index 1f56b39df94e601ee0fe2a34afaa915b5b1f5957..7197e9dd0a43ae6bb0b792d28af5f40066c5ef32 100644 (file)
--- a/doc/src/sgml/logicaldecoding.sgml
+++ b/doc/src/sgml/logicaldecoding.sgml
@@ -149,7 +149,7 @@ postgres=# SELECT pg_drop_replication_slot('regression_slot');
     another connection.
    
 
-# pg_recvlogical -d postgres --slot test --create
+# pg_recvlogical -d postgres --slot test --create-slot
 # pg_recvlogical -d postgres --slot test --start -f -
 CTRL-Z
 # psql -d postgres -c "INSERT INTO data(data) VALUES('4');"
@@ -158,7 +158,7 @@ BEGIN 693
 table public.data: INSERT: id[integer]:4 data[text]:'4'
 COMMIT 693
 CTRL-C
-# pg_recvlogical -d postgres --slot test --drop
+# pg_recvlogical -d postgres --slot test --drop-slot
 
   
   

diff --git a/doc/src/sgml/ref/pg_recvlogical.sgml b/doc/src/sgml/ref/pg_recvlogical.sgml
index 76240fe991a781c420fc94f8fd2cf70bd8d32514..f30b4844d7e85dccd9e7c162cd04f6ea75b6157b 100644 (file)
--- a/doc/src/sgml/ref/pg_recvlogical.sgml
+++ b/doc/src/sgml/ref/pg_recvlogical.sgml
@@ -53,7 +53,7 @@ PostgreSQL documentation
     
 
      
-      --create
+      --create-slot
       
        
         Create a new logical replication slot with the name specified in
@@ -82,7 +82,7 @@ PostgreSQL documentation
      
 
      
-      --drop
+      --drop-slot
       
        
         Drop the replication slot with the name specified
@@ -266,8 +266,9 @@ PostgreSQL documentation
       
        
         In --start mode, use the existing logical replication slot named
-        slot_name. In --create mode, create the
-        slot with this name. In --drop mode, delete the slot with this name.
+        slot_name. In --create-slot
+        mode, create the slot with this name. In --drop-slot
+        mode, delete the slot with this name.
        
       
      

diff --git a/src/bin/pg_basebackup/pg_recvlogical.c b/src/bin/pg_basebackup/pg_recvlogical.c
index c48ceccf90164597e235d64980382ef85b13baae..1a01167912306cb228000f9f8b37a79917ebc041 100644 (file)
--- a/src/bin/pg_basebackup/pg_recvlogical.c
+++ b/src/bin/pg_basebackup/pg_recvlogical.c
@@ -91,9 +91,9 @@ 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(_("      --create-slot      create a new replication slot (for the slot's name see --slot)\n"));
+   printf(_("      --drop-slot        drop the 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(_("      --drop             drop the replication slot (for the slot's name see --slot)\n"));
    printf(_("\nReport bugs to .\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}
    };
    int         c;
@@ -814,7 +814,7 @@ main(int argc, char **argv)
 
    if (do_drop_slot && (do_create_slot || do_start_slot))
    {
-       fprintf(stderr, _("%s: cannot use --create or --start together with --drop\n"), progname);
+       fprintf(stderr, _("%s: cannot use --create-slot or --start together with --drop-slot\n"), progname);
        fprintf(stderr, _("Try \"%s --help\" for more information.\n"),
                progname);
        exit(1);
@@ -822,7 +822,7 @@ main(int argc, char **argv)
 
    if (startpos != InvalidXLogRecPtr && (do_create_slot || do_drop_slot))
    {
-       fprintf(stderr, _("%s: cannot use --create or --drop together with --startpos\n"), progname);
+       fprintf(stderr, _("%s: cannot use --create-slot or --drop-slot together with --startpos\n"), progname);
        fprintf(stderr, _("Try \"%s --help\" for more information.\n"),
                progname);
        exit(1);