Skip to content
Snippets Groups Projects
Commit 93b0f630 authored by Vijay Pai's avatar Vijay Pai
Browse files

Merge pull request #1979 from ctiller/cereal-is-bad-for-your-health

Dont serialize new call requests under the server test lock
parents b6015c38 52d646ed
No related branches found
No related tags found
No related merge requests found
......@@ -101,10 +101,11 @@ class AsyncQpsServerTest : public Server {
ServerRpcContext *ctx = detag(got_tag);
// The tag is a pointer to an RPC context to invoke
bool still_going = ctx->RunNextState(ok);
std::lock_guard<std::mutex> g(shutdown_mutex_);
std::unique_lock<std::mutex> g(shutdown_mutex_);
if (!shutdown_) {
// this RPC context is done, so refresh it
if (!still_going) {
g.unlock();
ctx->Reset();
}
} else {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment