gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

Modify help msg PING_BULK to PING_MBULK in benchmark (#8109)

Browse Source 
As described in redis-benchamrk help message 'The test names are the same as the ones produced as output.', In redis-benchmark output, we can only see PING_BULK, but the cmd `redis-benchmark -t ping_bulk` is not supported. We  have to run it with ping_mbulk which is not user friendly.
This commit is contained in:
luhuachao 2020-12-02 19:17:25 +08:00 committed by GitHub
parent 69b7113bb5
commit 7885faf18b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/redis-benchmark.c
View File

@ -1829,7 +1829,7 @@ int main(int argc, const char **argv) {
if (test_is_selected("ping_mbulk") || test_is_selected("ping")) {
len = redisFormatCommand(&cmd,"PING");
benchmark("PING_BULK",cmd,len);
benchmark("PING_MBULK",cmd,len);
free(cmd);
}

2
tests/integration/redis-benchmark.tcl
View File

@ -33,7 +33,7 @@ start_server {tags {"benchmark"}} {
puts [colorstr red "redis-benchmark non zero code. first line: $first_line"]
fail "redis-benchmark non zero code. first line: $first_line"
}
# ping total calls are 2*issued commands per test due to PING_INLINE and PING_BULK
# ping total calls are 2*issued commands per test due to PING_INLINE and PING_MBULK
assert_match {*calls=200,*} [cmdstat ping]
assert_match {*calls=100,*} [cmdstat set]
assert_match {*calls=100,*} [cmdstat get]