Skip to content

Commit 5561596

Browse files
namsicjhpark816
authored andcommitted
CLEANUP: Call shutdown_server() in sigterm_handler()
1 parent c82e350 commit 5561596

File tree

1 file changed

+12
-21
lines changed

1 file changed

+12
-21
lines changed

memcached.c

Lines changed: 12 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -14524,6 +14524,17 @@ static void remove_pidfile(const char *pid_file)
1452414524
}
1452514525
}
1452614526

14527+
static void shutdown_server(void)
14528+
{
14529+
memcached_shutdown = 1;
14530+
14531+
#ifdef ENABLE_ZK_INTEGRATION
14532+
if (arcus_zk_cfg) {
14533+
arcus_zk_shutdown = 1;
14534+
}
14535+
#endif
14536+
}
14537+
1452714538
static void sigterm_handler(int sig)
1452814539
{
1452914540
assert(sig == SIGTERM || sig == SIGINT);
@@ -14533,13 +14544,7 @@ static void sigterm_handler(int sig)
1453314544
"memcached shutdown by signal(%s)\n",
1453414545
(sig == SIGINT ? "SIGINT" : "SIGTERM"));
1453514546
}
14536-
memcached_shutdown = 1;
14537-
14538-
#ifdef ENABLE_ZK_INTEGRATION
14539-
if (arcus_zk_cfg) {
14540-
arcus_zk_shutdown = 1;
14541-
}
14542-
#endif
14547+
shutdown_server();
1454314548
}
1454414549

1454514550
static int install_sigterm_handler(void)
@@ -14814,20 +14819,6 @@ static bool is_my_key(const char *key, size_t nkey)
1481414819
}
1481514820
#endif
1481614821

14817-
static void shutdown_server(void)
14818-
{
14819-
if (settings.verbose) {
14820-
mc_logger->log(EXTENSION_LOG_INFO, NULL, "memcached shutdown by api\n");
14821-
}
14822-
memcached_shutdown = 1;
14823-
14824-
#ifdef ENABLE_ZK_INTEGRATION
14825-
if (arcus_zk_cfg) {
14826-
arcus_zk_shutdown = 1;
14827-
}
14828-
#endif
14829-
}
14830-
1483114822
static EXTENSION_LOGGER_DESCRIPTOR* get_logger(void)
1483214823
{
1483314824
return settings.extensions.logger;

0 commit comments

Comments
 (0)