Github user dcelasun commented on the issue:
https://github.com/apache/thrift/pull/1459
It would be nice to add a client test using e.g `context.WithTimeout()`.
Something like:
- In the handler function, sleep for 100ms before returning
- In the client, call the RPC with
`context.WithTimeout(context.Background(), 50*time.Millisecond)`
- Check if `ctx.Err() == context.Canceled`
Otherwise LGTM.---
