go-redis-server

go-redis-server copied to clipboard

I found that fmt.Sscanf of go1.5 and go1.6 needs "*%d\r\n" for these newline terminated lines. I also fixed the typos in the Errorf().

tgrennan

A Bug found when execute " hset " command.

1

``` 127.0.0.1:6389> hset test good boy Could not connect to Redis at 127.0.0.1:6389: Connection refused ``` there some traceback logs from server-side: ``` panic: runtime error: assignment to entry in...

openinx

The older version think '*2\r\n' as invalid input when parsing multiline command. fmt.SScanf must indicate '\r\n' explicitly now.

xxr3376

My colleague found that this code do not work well in PIPELINE mode. The reason is that the old code create `bufio.Reader` for each request. When using pipeline mode, `bufio.Reader`...

xxr3376

I think the fmt.Sscanf or Readline implementation might have changed, but the server didn't work for me (go 1.5) until I changed the Sscanf to include \r\n. $go version go...

qzaidi

I don't know why run a goruntime at the begin of ServeClient.When clientChan close,the ChannelWriter will return,so that ChannelWriter.Channel did't work any more

wudikua

``` func readArgument(r *bufio.Reader) ([]byte, error) { line, err := r.ReadString('\n') if err != nil { return nil, malformed("$", line) } var argSize int if _, err := fmt.Sscanf(line, "$%d\r",...

lfyzjck

dim

Hi, How can I store connection related data ? like the current db for each connection. Currently, spawning two `redis-cli` console, and `select`ing a db in the first console, it...

tsileo