From ad7a407a13b87ec13d94a808111d2583bfd1d217 Mon Sep 17 00:00:00 2001 From: Amitay Isaacs Date: Thu, 17 Mar 2016 14:22:17 +1100 Subject: [PATCH] ctdb-recovery-helper: Introduce new #define variable ... instead of hardcoding number of retries. Signed-off-by: Amitay Isaacs Reviewed-by: Martin Schwenke --- ctdb/server/ctdb_recovery_helper.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ctdb/server/ctdb_recovery_helper.c b/ctdb/server/ctdb_recovery_helper.c index edcb3a39fd4..92dcdeb0421 100644 --- a/ctdb/server/ctdb_recovery_helper.c +++ b/ctdb/server/ctdb_recovery_helper.c @@ -36,6 +36,8 @@ static int recover_timeout = 120; +#define NUM_RETRIES 5 + #define TIMEOUT() timeval_current_ofs(recover_timeout, 0) static void LOG(const char *fmt, ...) @@ -1999,7 +2001,7 @@ static void db_recovery_one_done(struct tevent_req *subreq) } substate->num_fails += 1; - if (substate->num_fails < 5) { + if (substate->num_fails < NUM_RETRIES) { subreq = recover_db_send(state, state->ev, substate->client, substate->tun_list, substate->pnn_list, substate->count,