Todd Lipcon has submitted this change and it was merged. Change subject: cfile-test: use a faster data generator for 100M-string test ......................................................................
cfile-test: use a faster data generator for 100M-string test The current data generator using StringPrintf is not very fast, and since we don't care about the contents of the file for this test, it's just as easy to use FastHex64ToBuffer for better performance. This resulted in a ~2x speedup in the test. This means that the test will now better represent the underlying performance of the encoder and cfile writing code. Change-Id: I9578681aa2e064dbecd79fdce7b8a083ef03b3c4 Reviewed-on: http://gerrit.cloudera.org:8080/5201 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <danburk...@apache.org> --- M src/kudu/cfile/cfile-test.cc 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5201 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9578681aa2e064dbecd79fdce7b8a083ef03b3c4 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
