Todd Lipcon has posted comments on this change. Change subject: WIP: Integrate the request tracker with the client ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3080/1/src/kudu/rpc/retriable_rpc.h File src/kudu/rpc/retriable_rpc.h: Line 40: const scoped_refptr<RequestTracker>& request_tracker, seeing that this has to get plumbed in here makes me wonder if we should actually just attach the 'client ID' to the Messenger rather than the client object, so that everything needed is already "within" the RPC system? -- To view, visit http://gerrit.cloudera.org:8080/3080 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I94207c294452fcbdb3a7901fdb702674d47553ee Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
