database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-11-07 19:06:32 +03:00
Code Activity

Remove WaitPMResult enum in pg_createsubscriber

Browse Source 
A simple boolean suffices.  This is cosmetic, so no backpatch.

Author: Álvaro Herrera <alvherre@kurilemu.de>
Discussion: https://postgr.es/m/202510311750.mxiykx3tp4mx@alvherre.pgsql
This commit is contained in:
Álvaro Herrera
2025-11-03 12:59:32 +01:00
parent ad25744f43
commit f242dbcede
1 changed files with 3 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/bin/pg_basebackup/pg_createsubscriber.c
View File

@@ -156,12 +156,6 @@ static char *subscriber_dir = NULL;
static bool recovery_ended = false;
static bool standby_running = false;
enum WaitPMResult
{
POSTMASTER_READY,
POSTMASTER_STILL_STARTING
};
/*
* Cleanup objects that were created by pg_createsubscriber if there is an
@@ -1584,7 +1578,7 @@ static void
wait_for_end_recovery(const char *conninfo, const struct CreateSubscriberOptions *opt)
{
PGconn *conn;
int status = POSTMASTER_STILL_STARTING;
bool ready = false;
int timer = 0;
pg_log_info("waiting for the target server to reach the consistent state");
@@ -1596,7 +1590,7 @@ wait_for_end_recovery(const char *conninfo, const struct CreateSubscriberOptions
/* Did the recovery process finish? We're done if so. */
if (dry_run || !server_is_in_recovery(conn))
{
status = POSTMASTER_READY;
ready = true;
recovery_ended = true;
break;
}
@@ -1617,7 +1611,7 @@ wait_for_end_recovery(const char *conninfo, const struct CreateSubscriberOptions
disconnect_database(conn, false);
if (status == POSTMASTER_STILL_STARTING)
if (!ready)
pg_fatal("server did not end recovery");
pg_log_info("target server reached the consistent state");