David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/4826
Change subject: TODO cleanup in TransactionManager ...................................................................... TODO cleanup in TransactionManager This removes a couple of TODOs that were now wrong because we now have exactly-once retry semantics. The first was about not having to perform the actual waiting for commit wait on followers. With the new retry semantics this could easily mean that upon retrying a request on a follower a client would observe the result of the operation before it safe to do so. The second was about not peforming the waiting for commit wait if the RPCs deadline was expired. A client can now have a total deadline that is higher than a specific RPC's deadline and has exactly once semantics meaning even if the specific RPC's deadline is expired the client might still retry within the context of the global deadline. In that case we wouldn't want to have the client observe the operation before it was supposed to. Change-Id: Ib41a5a65dee29d26f0e7c72b4f2d30f1267c8cf2 --- M src/kudu/tablet/transactions/transaction_driver.cc 1 file changed, 2 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/26/4826/1 -- To view, visit http://gerrit.cloudera.org:8080/4826 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib41a5a65dee29d26f0e7c72b4f2d30f1267c8cf2 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <[email protected]>
