aboutsummaryrefslogtreecommitdiffhomepage
path: root/include
diff options
context:
space:
mode:
Diffstat (limited to 'include')
-rw-r--r--include/grpcpp/impl/codegen/client_unary_call.h7
1 files changed, 6 insertions, 1 deletions
diff --git a/include/grpcpp/impl/codegen/client_unary_call.h b/include/grpcpp/impl/codegen/client_unary_call.h
index b1c80764f2..18ee5b34bf 100644
--- a/include/grpcpp/impl/codegen/client_unary_call.h
+++ b/include/grpcpp/impl/codegen/client_unary_call.h
@@ -75,7 +75,12 @@ class BlockingUnaryCallImpl {
"No message returned for unary request");
}
} else {
- GPR_CODEGEN_ASSERT(!status_.ok());
+ // Some of the ops failed. For example, this can happen if deserialization
+ // of the message fails. gRPC Core guarantees that the op
+ // GRPC_OP_RECV_STATUS_ON_CLIENT always succeeds, so status would still be
+ // filled.
+ // TODO(yashykt): If deserialization fails, but the status received is OK,
+ // then it might be a good idea to change the status to reflect this.
}
}
Status status() { return status_; }