David Ribeiro Alves has posted comments on this change.

Change subject: Add a request tracker to track client rpc sequence numbers
......................................................................


Patch Set 4:

thanks for the review todd.
I was thinking of using the time bounds when the server stops hearing from the 
client. In the paper they have the leases for that, but we'd use just the time 
bounds. Sending the lower bound seems prudent as keeping all responses for 
something like 10 minutes can translate to a lot of memory.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I23201625ca02f244dc94205d88dabc01608de471
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to