Unverified Commit 317464a3 authored by chenyang8094's avatar chenyang8094 Committed by GitHub
Browse files

Fix failing test due to recent change in transaction propagation (#10006)



PR #9890 may have introduced a problem.
There are tests that use MULTI-EXEC to make sure two BGSAVE / BGREWRITEAOF are executed together.
But now it's not valid to run run commands that create a snapshot inside a transaction (gonna be blocked soon)
This PR modifies the test not to rely on MULTI-EXEC.
Co-authored-by: default avatarOran Agra <oran@redislabs.com>
parent 0f15e025
...@@ -179,28 +179,28 @@ start_server {tags {"aofrw external:skip"} overrides {aof-use-rdb-preamble no}} ...@@ -179,28 +179,28 @@ start_server {tags {"aofrw external:skip"} overrides {aof-use-rdb-preamble no}}
} }
test {BGREWRITEAOF is delayed if BGSAVE is in progress} { test {BGREWRITEAOF is delayed if BGSAVE is in progress} {
r multi r flushall
r set k v
r config set rdb-key-save-delay 10000000
r bgsave r bgsave
r bgrewriteaof assert_match {*scheduled*} [r bgrewriteaof]
r info persistence assert_equal [s aof_rewrite_scheduled] 1
set res [r exec] r config set rdb-key-save-delay 0
assert_match {*scheduled*} [lindex $res 1] catch {exec kill -9 [get_child_pid 0]}
assert_match {*aof_rewrite_scheduled:1*} [lindex $res 2] while {[s aof_rewrite_scheduled] eq 1} {
while {[string match {*aof_rewrite_scheduled:1*} [r info persistence]]} {
after 100 after 100
} }
} }
test {BGREWRITEAOF is refused if already in progress} { test {BGREWRITEAOF is refused if already in progress} {
r config set aof-use-rdb-preamble yes
r config set rdb-key-save-delay 10000000
catch { catch {
r multi
r bgrewriteaof r bgrewriteaof
r bgrewriteaof r bgrewriteaof
r exec
} e } e
assert_match {*ERR*already*} $e assert_match {*ERR*already*} $e
while {[string match {*aof_rewrite_scheduled:1*} [r info persistence]]} { r config set rdb-key-save-delay 0
after 100 catch {exec kill -9 [get_child_pid 0]}
}
} }
} }
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