From 55bb5bdf803ffcb6e643d35611af4938ee29fd1c Mon Sep 17 00:00:00 2001
From: Vijay Pai <vpai@google.com>
Date: Mon, 23 Mar 2015 12:44:09 -0700
Subject: [PATCH] No need to do an extra read

---
 test/cpp/qps/client_sync.cc | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/test/cpp/qps/client_sync.cc b/test/cpp/qps/client_sync.cc
index 1e14aa85c5..e4ee45a72d 100644
--- a/test/cpp/qps/client_sync.cc
+++ b/test/cpp/qps/client_sync.cc
@@ -108,13 +108,10 @@ class SynchronousStreamingClient GRPC_FINAL : public SynchronousClient {
     if (stream_) {
       SimpleResponse response;
       stream_->WritesDone();
-      EXPECT_FALSE(stream_->Read(&response));
-
-      Status s = stream_->Finish();
-      EXPECT_TRUE(s.IsOk());
+      EXPECT_TRUE(stream_->Finish().IsOk());
     }
   }
-  
+
   void ThreadFunc(Histogram* histogram, size_t thread_idx) GRPC_OVERRIDE {
     double start = Timer::Now();
     EXPECT_TRUE(stream_->Write(request_));
@@ -123,7 +120,8 @@ class SynchronousStreamingClient GRPC_FINAL : public SynchronousClient {
   }
   private:
     grpc::ClientContext context_;
-    std::unique_ptr<grpc::ClientReaderWriter<SimpleRequest,SimpleResponse>> stream_;
+    std::unique_ptr<grpc::ClientReaderWriter<SimpleRequest,
+                                             SimpleResponse>> stream_;
 };
 
 std::unique_ptr<Client>
-- 
GitLab