andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

db_bench: fix "micros/op" reporting (#4949)

Browse Source 
Summary:
4985a9f73b (diff-e5276985b26a0551957144f4420a594bR511)
changes the meaning of latency reporting from running time per query, to elapse_time / #ops, without providing a reason why.
Considering that this is a counter-intuitive reporting, Reverting the change.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/4949

Differential Revision: D13964684

Pulled By: siying

fbshipit-source-id: d6304d3d4b5a802daa292302623c7dbca9a680bc
This commit is contained in:
Siying Dong 2019-02-05 17:15:42 -08:00 committed by Facebook Github Bot
parent 71cae59a99
commit d9c9f3c809
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/db_bench_tool.cc
View File

@ -1831,7 +1831,7 @@ class Stats {
fprintf(stdout, "%-12s : %11.3f micros/op %ld ops/sec;%s%s\n", fprintf(stdout, "%-12s : %11.3f micros/op %ld ops/sec;%s%s\n",
name.ToString().c_str(), name.ToString().c_str(),
elapsed * 1e6 / done_, seconds_ * 1e6 / done_,
(long)throughput, (long)throughput,
(extra.empty() ? "" : " "), (extra.empty() ? "" : " "),
extra.c_str()); extra.c_str());