Commit 12626ce9 authored by Oran Agra's avatar Oran Agra Committed by antirez
Browse files

fix race in module api test for fork

in some cases we were trying to kill the fork before it got created
parent 2ecab0b6
...@@ -42,7 +42,7 @@ int fork_create(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) ...@@ -42,7 +42,7 @@ int fork_create(RedisModuleCtx *ctx, RedisModuleString **argv, int argc)
/* child */ /* child */
RedisModule_Log(ctx, "notice", "fork child started"); RedisModule_Log(ctx, "notice", "fork child started");
usleep(200000); usleep(500000);
RedisModule_Log(ctx, "notice", "fork child exiting"); RedisModule_Log(ctx, "notice", "fork child exiting");
RedisModule_ExitFromChild(code_to_exit_with); RedisModule_ExitFromChild(code_to_exit_with);
/* unreachable */ /* unreachable */
......
...@@ -20,9 +20,8 @@ start_server {tags {"modules"}} { ...@@ -20,9 +20,8 @@ start_server {tags {"modules"}} {
test {Module fork kill} { test {Module fork kill} {
r fork.create 3 r fork.create 3
after 20 after 250
r fork.kill r fork.kill
after 100
assert {[count_log_message "fork child started"] eq "2"} assert {[count_log_message "fork child started"] eq "2"}
assert {[count_log_message "Received SIGUSR1 in child"] eq "1"} assert {[count_log_message "Received SIGUSR1 in child"] eq "1"}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment