Skip to content
This repository has been archived by the owner on Jan 21, 2022. It is now read-only.

Add support for CONFIG GET #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shawwn
Copy link

@shawwn shawwn commented Nov 15, 2020

On redis-cli version 6.0.6, ./benchmark.sh produces warnings like:

$ ./benchmark.sh 
pydis 1
Could not connect to Redis at 127.0.0.1:7878: Connection refused
WARN: could not fetch server CONFIG
Could not connect to Redis at 127.0.0.1:7878: setsockopt(TCP_NODELAY): Invalid argument
pydis 2
Could not connect to Redis at 127.0.0.1:7878: Connection refused
WARN: could not fetch server CONFIG
Could not connect to Redis at 127.0.0.1:7878: setsockopt(TCP_NODELAY): Invalid argument
pydis 3
Could not connect to Redis at 127.0.0.1:7878: Connection refused
WARN: could not fetch server CONFIG
Could not connect to Redis at 127.0.0.1:7878: setsockopt(TCP_NODELAY): Invalid argument
redis 1
Could not connect to Redis at 127.0.0.1:6379: Connection refused
WARN: could not fetch server CONFIG
All clients disconnected... aborting.

This PR implements support for a very basic, hardcoded server config that squelches this warning.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant