song bruce zhang has posted comments on this change.

Change subject: rpc: avoid crashing if the server generates an invalid RPC 
response
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2914/2/src/kudu/rpc/inbound_call.cc
File src/kudu/rpc/inbound_call.cc:

Line 142:   
Path seems to have not touch method "SerializeResponseTo" .
In  my fatal log, there is a check failed 
"F0429 20:05:16.809551 24360 inbound_call.cc:167] Check failed: 
response_hdr_buf_.size() > 0 (0 vs. 0) "
Don't need to remove this check?


-- 
To view, visit http://gerrit.cloudera.org:8080/2914
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7845388b1f520158e554548d668836d40b10ca6
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: song bruce zhang <[email protected]>
Gerrit-HasComments: Yes

Reply via email to