Netflix / dynomite

A generic dynamo implementation for different k-v storage engines
Apache License 2.0
4.19k stars 531 forks source link

Support the Redis "QUIT" command properly #729

Closed smukil closed 4 years ago

smukil commented 4 years ago

The QUIT command so far was handled by just closing the connection on the server side and did not respect the contract of the QUIT command.

The Redis documentation states that the QUIT command must return "+OK\r\n" back to the client so that the client can close the connection on its side.

This patch takes advantage of simulating a datastore response from the previous patch to achieve this.