David Ribeiro Alves has posted comments on this change. Change subject: Add a request tracker to track client rpc sequence numbers ......................................................................
Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3078/2/src/kudu/rpc/request_tracker.cc File src/kudu/rpc/request_tracker.cc: Line 35: if (PREDICT_FALSE(incomplete_rpcs_.size() >= FLAGS_rpc_max_in_flight_linearizable_rpcs)) { > what's the purpose of this bound? it's just to bound the size of the set? S no, this is from the paper and it's meant to bound the number of outstanding "tracked" rpcs per client so that we can further bound the amount of state in the servers -- 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: 2 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: Yes
