From eb14f4298cc69f27a5b093549e8eeb4cabec3d2c Mon Sep 17 00:00:00 2001 From: zliang Date: Fri, 18 Aug 2023 10:20:00 -0600 Subject: [PATCH] have emit system free memory within metrics_time_taken_us and also add metric time taken for it --- src/modules/keydb_modstatsd/modmain.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/modules/keydb_modstatsd/modmain.cpp b/src/modules/keydb_modstatsd/modmain.cpp index b40fc3c76..011d6108e 100644 --- a/src/modules/keydb_modstatsd/modmain.cpp +++ b/src/modules/keydb_modstatsd/modmain.cpp @@ -614,6 +614,10 @@ void event_cron_handler(struct RedisModuleCtx *ctx, RedisModuleEvent eid, uint64 // g_stats->timing("handle_client_info_time_taken_us", ustime() - commandStartTime); // RedisModule_FreeCallReply(reply); + commandStartTime = ustime(); + emit_system_free_memory(); + g_stats->timing("emit_free_system_memory_time_taken_us", ustime() - commandStartTime); + /* Log Keys */ reply = RedisModule_Call(ctx, "dbsize", ""); long long keys = RedisModule_CallReplyInteger(reply); @@ -621,8 +625,6 @@ void event_cron_handler(struct RedisModuleCtx *ctx, RedisModuleEvent eid, uint64 g_stats->gauge("keys", keys); RedisModule_Log(ctx, REDISMODULE_LOGLEVEL_DEBUG, "Emitting metric \"keys\": %llu", keys); g_stats->timing("metrics_time_taken_us", ustime() - startTime); - - emit_system_free_memory(); lastTime = curTime; }