github/repmgr
1
0
Fork 0
mirror of https://github.com/EnterpriseDB/repmgr.git synced 2026-03-25 08:06:29 +00:00
Code Issues Packages Projects Releases 1 Wiki Activity

repmgrd: rename some variables to better match the system functions they're populated from

Browse Source
This commit is contained in:
Ian Barwick
2016-04-08 15:46:55 +09:00
parent de573edaaa
commit 52281fcde8
1 changed files with 45 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

79
repmgrd.c
View File

@@ -696,15 +696,15 @@ standby_monitor(void)
PGresult *res; PGresult *res;
char monitor_standby_timestamp[MAXLEN]; char monitor_standby_timestamp[MAXLEN];
char last_wal_master_location[MAXLEN]; char last_wal_master_location[MAXLEN];
char last_wal_standby_received[MAXLEN]; char last_xlog_receive_location[MAXLEN];
char last_wal_standby_applied[MAXLEN]; char last_xlog_replay_location[MAXLEN];
char last_wal_standby_applied_timestamp[MAXLEN]; char last_xact_replay_timestamp[MAXLEN];
bool last_wal_standby_received_gte_replayed; bool last_xlog_receive_location_gte_replayed;
char sqlquery[QUERY_STR_LEN]; char sqlquery[QUERY_STR_LEN];
XLogRecPtr lsn_master; XLogRecPtr lsn_master_current_xlog_location;
XLogRecPtr lsn_standby_received; XLogRecPtr lsn_last_xlog_receive_location;
XLogRecPtr lsn_standby_applied; XLogRecPtr lsn_last_xlog_replay_location;
int connection_retries, int connection_retries,
ret; ret;
@@ -999,10 +999,10 @@ standby_monitor(void)
} }
strncpy(monitor_standby_timestamp, PQgetvalue(res, 0, 0), MAXLEN); strncpy(monitor_standby_timestamp, PQgetvalue(res, 0, 0), MAXLEN);
strncpy(last_wal_standby_received, PQgetvalue(res, 0, 1), MAXLEN); strncpy(last_xlog_receive_location, PQgetvalue(res, 0, 1), MAXLEN);
strncpy(last_wal_standby_applied, PQgetvalue(res, 0, 2), MAXLEN); strncpy(last_xlog_replay_location, PQgetvalue(res, 0, 2), MAXLEN);
strncpy(last_wal_standby_applied_timestamp, PQgetvalue(res, 0, 3), MAXLEN); strncpy(last_xact_replay_timestamp, PQgetvalue(res, 0, 3), MAXLEN);
last_wal_standby_received_gte_replayed = (strcmp(PQgetvalue(res, 0, 4), "t") == 0) last_xlog_receive_location_gte_replayed = (strcmp(PQgetvalue(res, 0, 4), "t") == 0)
? true ? true
: false; : false;
@@ -1011,13 +1011,13 @@ standby_monitor(void)
/* /*
* In the unusual event of a standby becoming disconnected from the primary, * In the unusual event of a standby becoming disconnected from the primary,
* while this repmgrd remains connected to the primary, subtracting * while this repmgrd remains connected to the primary, subtracting
* "lsn_standby_applied" from "lsn_standby_received" and coercing to * "last_xlog_replay_location" from "lsn_last_xlog_receive_location" and coercing to
* (long long unsigned int) will result in a meaningless, very large * (long long unsigned int) will result in a meaningless, very large
* value which will overflow a BIGINT column and spew error messages into the * value which will overflow a BIGINT column and spew error messages into the
* PostgreSQL log. In the absence of a better strategy, skip attempting * PostgreSQL log. In the absence of a better strategy, skip attempting
* to insert a monitoring record. * to insert a monitoring record.
*/ */
if (last_wal_standby_received_gte_replayed == false) if (last_xlog_receive_location_gte_replayed == false)
{ {
log_verbose(LOG_WARNING, log_verbose(LOG_WARNING,
"Invalid replication_lag value calculated - is this standby connected to its upstream?\n"); "Invalid replication_lag value calculated - is this standby connected to its upstream?\n");
@@ -1039,29 +1039,40 @@ standby_monitor(void)
PQclear(res); PQclear(res);
/* Calculate the lag */ /* Calculate the lag */
lsn_master = lsn_to_xlogrecptr(last_wal_master_location, NULL); lsn_master_current_xlog_location = lsn_to_xlogrecptr(last_wal_master_location, NULL);
lsn_standby_received = lsn_to_xlogrecptr(last_wal_standby_received, NULL); lsn_last_xlog_receive_location = lsn_to_xlogrecptr(last_xlog_receive_location, NULL);
lsn_standby_applied = lsn_to_xlogrecptr(last_wal_standby_applied, NULL); lsn_last_xlog_replay_location = lsn_to_xlogrecptr(last_xlog_replay_location, NULL);
/* /*
* Build the SQL to execute on master * Build the SQL to execute on master
*/ */
sqlquery_snprintf(sqlquery, sqlquery_snprintf(sqlquery,
"INSERT INTO %s.repl_monitor " "INSERT INTO %s.repl_monitor "
" (primary_node, standby_node, " " (primary_node, "
" last_monitor_time, last_apply_time, " " standby_node, "
" last_wal_primary_location, last_wal_standby_location, " " last_monitor_time, "
" replication_lag, apply_lag ) " " last_apply_time, "
" VALUES(%d, %d, " " last_wal_primary_location, "
" '%s'::TIMESTAMP WITH TIME ZONE, '%s'::TIMESTAMP WITH TIME ZONE, " " last_wal_standby_location, "
" '%s', '%s', " " replication_lag, "
" %llu, %llu) ", " apply_lag ) "
" VALUES(%d, "
" %d, "
" '%s'::TIMESTAMP WITH TIME ZONE, "
" '%s'::TIMESTAMP WITH TIME ZONE, "
" '%s', "
" '%s', "
" %llu, "
" %llu) ",
get_repmgr_schema_quoted(master_conn), get_repmgr_schema_quoted(master_conn),
master_options.node, local_options.node, master_options.node,
monitor_standby_timestamp, last_wal_standby_applied_timestamp, local_options.node,
last_wal_master_location, last_wal_standby_received, monitor_standby_timestamp,
(long long unsigned int)(lsn_master - lsn_standby_received), last_xact_replay_timestamp,
(long long unsigned int)(lsn_standby_received - lsn_standby_applied)); last_wal_master_location,
last_xlog_receive_location,
(long long unsigned int)(lsn_master_current_xlog_location - lsn_last_xlog_receive_location),
(long long unsigned int)(lsn_last_xlog_receive_location - lsn_last_xlog_replay_location));
/* /*
* Execute the query asynchronously, but don't check for a result. We will * Execute the query asynchronously, but don't check for a result. We will
@@ -1099,7 +1110,7 @@ do_master_failover(void)
XLogRecPtr xlog_recptr; XLogRecPtr xlog_recptr;
bool lsn_format_ok; bool lsn_format_ok;
char last_wal_standby_applied[MAXLEN]; char last_xlog_replay_location[MAXLEN];
PGconn *node_conn = NULL; PGconn *node_conn = NULL;
@@ -1282,8 +1293,8 @@ do_master_failover(void)
" considered as new master and exit.\n"), " considered as new master and exit.\n"),
PQerrorMessage(my_local_conn)); PQerrorMessage(my_local_conn));
PQclear(res); PQclear(res);
sprintf(last_wal_standby_applied, "'%X/%X'", 0, 0); sprintf(last_xlog_replay_location, "'%X/%X'", 0, 0);
update_shared_memory(last_wal_standby_applied); update_shared_memory(last_xlog_replay_location);
terminate(ERR_DB_QUERY); terminate(ERR_DB_QUERY);
} }
/* write last location in shared memory */ /* write last location in shared memory */
@@ -2158,7 +2169,7 @@ terminate(int retval)
static void static void
update_shared_memory(char *last_wal_standby_applied) update_shared_memory(char *last_xlog_replay_location)
{ {
PGresult *res; PGresult *res;
char sqlquery[QUERY_STR_LEN]; char sqlquery[QUERY_STR_LEN];
@@ -2166,7 +2177,7 @@ update_shared_memory(char *last_wal_standby_applied)
sprintf(sqlquery, sprintf(sqlquery,
"SELECT %s.repmgr_update_standby_location('%s')", "SELECT %s.repmgr_update_standby_location('%s')",
get_repmgr_schema_quoted(my_local_conn), get_repmgr_schema_quoted(my_local_conn),
last_wal_standby_applied); last_xlog_replay_location);
/* If an error happens, just inform about that and continue */ /* If an error happens, just inform about that and continue */
res = PQexec(my_local_conn, sqlquery); res = PQexec(my_local_conn, sqlquery);