Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15150
Change subject: [consensus] cleanup on TimeManager ...................................................................... [consensus] cleanup on TimeManager A tiny cleanup: * removed outdated TODO TimeManager::WaitUntilSafe() * removed questinable PREDICT_FALSE() and its extra 'if ()' scope I also thought of replacing std::vector with std::deque as the container for TimeManager::waiters_, but I'm not sure it's worth it given the expected number of elements in there. Change-Id: I1315e058b4a8e0518c96fad6454aeb4d0a46b2e6 --- M src/kudu/consensus/time_manager.cc M src/kudu/consensus/time_manager.h 2 files changed, 9 insertions(+), 14 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/15150/1 -- To view, visit http://gerrit.cloudera.org:8080/15150 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I1315e058b4a8e0518c96fad6454aeb4d0a46b2e6 Gerrit-Change-Number: 15150 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
