aboutsummaryrefslogtreecommitdiffhomepage
path: root/test/cpp
diff options
context:
space:
mode:
authorGravatar Yuchen Zeng <zyc@google.com>2016-09-23 14:14:18 -0700
committerGravatar Yuchen Zeng <zyc@google.com>2016-09-23 14:14:18 -0700
commit27170357abbb9609b9f0ac47c1926a7eb383ca67 (patch)
tree960fb7039b8f738589e70fcca15fb9bbf0720e7a /test/cpp
parent942c264861dedd8020fc18d65933e8f4f57e3e46 (diff)
Improve ProtoReflectionDescriptorDatabase
Diffstat (limited to 'test/cpp')
-rw-r--r--test/cpp/util/proto_reflection_descriptor_database.cc9
-rw-r--r--test/cpp/util/proto_reflection_descriptor_database.h2
2 files changed, 7 insertions, 4 deletions
diff --git a/test/cpp/util/proto_reflection_descriptor_database.cc b/test/cpp/util/proto_reflection_descriptor_database.cc
index f0d14c686a..ae633ea7f4 100644
--- a/test/cpp/util/proto_reflection_descriptor_database.cc
+++ b/test/cpp/util/proto_reflection_descriptor_database.cc
@@ -314,13 +314,16 @@ ProtoReflectionDescriptorDatabase::GetStream() {
return stream_;
}
-void ProtoReflectionDescriptorDatabase::DoOneRequest(
+bool ProtoReflectionDescriptorDatabase::DoOneRequest(
const ServerReflectionRequest& request,
ServerReflectionResponse& response) {
+ bool request_succeed = false;
stream_mutex_.lock();
- GetStream()->Write(request);
- GetStream()->Read(&response);
+ if (GetStream()->Write(request) && GetStream()->Read(&response)) {
+ request_succeed = true;
+ }
stream_mutex_.unlock();
+ return request_succeed;
}
} // namespace grpc
diff --git a/test/cpp/util/proto_reflection_descriptor_database.h b/test/cpp/util/proto_reflection_descriptor_database.h
index 0e69696d5f..54114f1081 100644
--- a/test/cpp/util/proto_reflection_descriptor_database.h
+++ b/test/cpp/util/proto_reflection_descriptor_database.h
@@ -111,7 +111,7 @@ class ProtoReflectionDescriptorDatabase : public protobuf::DescriptorDatabase {
const std::shared_ptr<ClientStream> GetStream();
- void DoOneRequest(
+ bool DoOneRequest(
const grpc::reflection::v1alpha::ServerReflectionRequest& request,
grpc::reflection::v1alpha::ServerReflectionResponse& response);