pg_createsubscriber: Fix error complaining about the wrong thing
authorÁlvaro Herrera <[email protected]>
Fri, 31 Oct 2025 16:43:15 +0000 (17:43 +0100)
committerÁlvaro Herrera <[email protected]>
Fri, 31 Oct 2025 16:43:15 +0000 (17:43 +0100)
The code updates the system identifier, then runs pg_walreset; if the
latter fails, it complains about the former, which makes no sense.
Change the error message to complain about the right thing.

Noticed while reviewing a patch touching nearby code.

Author: Álvaro Herrera <[email protected]>
Backpatch-through: 17

src/bin/pg_basebackup/pg_createsubscriber.c

index 067a04ec6c329c5e66236f3c256999d87bc3ce16..95c96d205f1c6a4e8bd3b252bad26a21695535dc 100644 (file)
@@ -668,9 +668,9 @@ modify_subscriber_sysid(const struct CreateSubscriberOptions *opt)
        int         rc = system(cmd_str);
 
        if (rc == 0)
-           pg_log_info("subscriber successfully changed the system identifier");
+           pg_log_info("subscriber successfully reset WAL on the subscriber");
        else
-           pg_fatal("could not change system identifier of subscriber: %s", wait_result_to_str(rc));
+           pg_fatal("could not reset WAL on subscriber: %s", wait_result_to_str(rc));
    }
 
    pg_free(cf);