Additional logging for proto deserialization (#26159)

* Fix build failures in xds_end2end_test

* Change Status::OK to StatusCode::OK

* Added additional logging for proto deserialization failure

* Added additional logging for serialization failure

* Revert "Added additional logging for serialization failure"

This reverts commit 2dbbbed9d4.

* Revert "Added additional logging for proto deserialization failure"

This reverts commit 31f4e6e3c8.

* Move error logging for proto deserialization to server.cc

* Updated from GPR_ERROR to GPR_DEBUG
pull/26225/head
Alisha Nanda 4 years ago committed by GitHub
parent 7080182ae9
commit cbcb0253a0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 7
      src/cpp/server/server_cc.cc

@ -417,7 +417,9 @@ class Server::SyncRequest final : public grpc::internal::CompletionQueueTag {
: server_->resource_exhausted_handler_.get();
deserialized_request_ = handler->Deserialize(call_, request_payload_,
&request_status_, nullptr);
if (!request_status_.ok()) {
gpr_log(GPR_DEBUG, "Failed to deserialize message.");
}
request_payload_ = nullptr;
interceptor_methods_.AddInterceptionHookPoint(
grpc::experimental::InterceptionHookPoints::POST_RECV_MESSAGE);
@ -657,6 +659,9 @@ class Server::CallbackRequest final
req_->request_ = req_->method_->handler()->Deserialize(
req_->call_, req_->request_payload_, &req_->request_status_,
&req_->handler_data_);
if (!(req_->request_status_.ok())) {
gpr_log(GPR_DEBUG, "Failed to deserialize message.");
}
req_->request_payload_ = nullptr;
req_->interceptor_methods_.AddInterceptionHookPoint(
grpc::experimental::InterceptionHookPoints::POST_RECV_MESSAGE);

Loading…
Cancel
Save