search

tumblr / colossus

I/O and Microservice library for Scala
Apache License 2.0
1.14k stars 96 forks source link

Memcached server error #511

Closed benblack86 closed 7 years ago

benblack86 commented 7 years ago

Memcached can send a valid server error response and then disconnect:

Success(ErrorReply(...))

If using the load balancer with retries and the client had just disconnected the request would have been retrieved. However, since memcached returned a valid response, no retry logic is applied. Would be nice if a server error response resulted in a retry - maybe make it configurable and on by default.

benblack86 commented 7 years ago

Will think about this as part of service client changes.