Todd Lipcon has submitted this change and it was merged. Change subject: log: improve mt-log-test benchmark ......................................................................
log: improve mt-log-test benchmark - builds the entries to be appended outside of holding any lock, which is more realistic. The entries get their IDs set once inside the lock (similar to what we do in real code). - use the same higher-level APIs that the real append paths use - make the log segment size overridable - allow the "verification" step to be disabled to serve as a better benchmark - allow the concurrent "reader" thread to be disabled - a couple bug fixes here and there Change-Id: I870dc26e2937e7c92e3f0530e2c2880178507f12 Reviewed-on: http://gerrit.cloudera.org:8080/6283 Tested-by: Kudu Jenkins Reviewed-by: David Ribeiro Alves <dral...@apache.org> --- M src/kudu/consensus/mt-log-test.cc 1 file changed, 77 insertions(+), 65 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6283 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I870dc26e2937e7c92e3f0530e2c2880178507f12 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>