• alonre24's avatar
    redis-benchmark - add the support for binary strings (#9414) · 044e29dd
    alonre24 authored
    
    
    Recently, the option of sending an argument from stdin using `-x` flag
    was added to redis-benchmark (this option is available in redis-cli as well).
    However, using the `-x` option for sending a blobs that contains null-characters
    doesn't work as expected - the argument is trimmed in the first occurrence of
    `\X00` (unlike in redis-cli).  
    This PR aims to fix this issue and add the support for every binary string input,
    by sending arguments length to `redisFormatCommandArgv` when processing
    redis-benchmark command, so we won't treat the arguments as C-strings.
    
    Additionally, we add a simple test coverage for `-x` (without binary strings, and
    also remove an excessive server started in tests, and make sure to select db 0
    so that `r` and the benchmark work on the same db.
    Co-authored-by: default avatarOran Agra <oran@redislabs.com>
    044e29dd
redis-benchmark.c 78.7 KB