Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17866 )
Change subject: [rpc] Expose KRPC call_id to client application ...................................................................... [rpc] Expose KRPC call_id to client application KRPC assign call_id number for each RPC call at Request header. This call_id then sent back in the Response and allows to match it to the original Request. KRPC client, such as Impala, can utilize this call_id for tracing or logging slow RPC. This patch expose call_id through RpcController and RpcContext class for such purpose. Testing: - Add TestCallId in rpc-test.cc. - Add call_id check in CalculatorService::Add(). - Run and pass rpc-test. Change-Id: If20114ef2b416ed9b39277e90639a6277b226fbb Reviewed-on: http://gerrit.cloudera.org:8080/17866 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/rpc/rpc-test-base.h M src/kudu/rpc/rpc-test.cc M src/kudu/rpc/rpc_context.cc M src/kudu/rpc/rpc_context.h M src/kudu/rpc/rpc_controller.cc M src/kudu/rpc/rpc_controller.h 6 files changed, 47 insertions(+), 1 deletion(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17866 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If20114ef2b416ed9b39277e90639a6277b226fbb Gerrit-Change-Number: 17866 Gerrit-PatchSet: 4 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Tidy Bot (241)
