Unverified Commit d5ca72e3 authored by menwen's avatar menwen Committed by GitHub
Browse files

fix defrag test looking at the wrong latency metric (#9723)

the latency event was renamed in #7726, and the outcome was that the test was
ineffective (unable to measure the max latency, always seeing 0)
parent 526cbb5c
...@@ -130,7 +130,7 @@ start_server {tags {"defrag external:skip"} overrides {appendonly yes auto-aof-r ...@@ -130,7 +130,7 @@ start_server {tags {"defrag external:skip"} overrides {appendonly yes auto-aof-r
set max_latency 0 set max_latency 0
foreach event [r latency latest] { foreach event [r latency latest] {
lassign $event eventname time latency max lassign $event eventname time latency max
if {$eventname == "loading-cron"} { if {$eventname == "while-blocked-cron"} {
set max_latency $max set max_latency $max
} }
} }
...@@ -141,7 +141,7 @@ start_server {tags {"defrag external:skip"} overrides {appendonly yes auto-aof-r ...@@ -141,7 +141,7 @@ start_server {tags {"defrag external:skip"} overrides {appendonly yes auto-aof-r
puts "misses: $misses" puts "misses: $misses"
puts "max latency $max_latency" puts "max latency $max_latency"
puts [r latency latest] puts [r latency latest]
puts [r latency history loading-cron] puts [r latency history "while-blocked-cron"]
} }
# make sure we had defrag hits during AOF loading # make sure we had defrag hits during AOF loading
assert {$hits > 100000} assert {$hits > 100000}
......
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