diff options
author | Yuchen Zeng <zyc@google.com> | 2016-09-23 14:14:18 -0700 |
---|---|---|
committer | Yuchen Zeng <zyc@google.com> | 2016-09-23 14:14:18 -0700 |
commit | 27170357abbb9609b9f0ac47c1926a7eb383ca67 (patch) | |
tree | 960fb7039b8f738589e70fcca15fb9bbf0720e7a /test/cpp | |
parent | 942c264861dedd8020fc18d65933e8f4f57e3e46 (diff) |
Improve ProtoReflectionDescriptorDatabase
Diffstat (limited to 'test/cpp')
-rw-r--r-- | test/cpp/util/proto_reflection_descriptor_database.cc | 9 | ||||
-rw-r--r-- | test/cpp/util/proto_reflection_descriptor_database.h | 2 |
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); |