diff options
author | Kyle Zhang <kyle@smartx.com> | 2019-04-25 09:44:07 +0800 |
---|---|---|
committer | Kyle Zhang <kyle@smartx.com> | 2019-04-25 09:54:05 +0800 |
commit | d3d1c8a0f40a7eaa12a5bb702fa01786b7c3a646 (patch) | |
tree | 83d0d84f23b7e5421eabb68d08029fc9339af939 /include | |
parent | 3dc9202f78a3eb30ee8c0267e4e4be2e3f986e45 (diff) | |
download | leveldb-d3d1c8a0f40a7eaa12a5bb702fa01786b7c3a646.tar.gz |
don't check current key in DBIter::Next()
When iter_ is pointing to current key, we can safely move to the next
key to avoid checking current key, which is of course not necessary.
Benchmark shows that 'readseq' has about 8% performance improvement.
Without patch:
>./db_bench --benchmarks=readseq --num=$((4<<20)) --db=/tmp/db --use_existing_db=1
LevelDB: version 1.21
Date: Thu Apr 25 09:37:21 2019
CPU: 32 * Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz
CPUCache: 20480 KB
Keys: 16 bytes each
Values: 100 bytes each (50 bytes after compression)
Entries: 4194304
RawSize: 464.0 MB (estimated)
FileSize: 264.0 MB (estimated)
------------------------------------------------
readseq : 0.196 micros/op; 565.7 MB/s
With patch:
>./db_bench --benchmarks=readseq --num=$((4<<20)) --db=/tmp/db --use_existing_db=1
LevelDB: version 1.21
Date: Thu Apr 25 09:38:20 2019
CPU: 32 * Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz
CPUCache: 20480 KB
Keys: 16 bytes each
Values: 100 bytes each (50 bytes after compression)
Entries: 4194304
RawSize: 464.0 MB (estimated)
FileSize: 264.0 MB (estimated)
------------------------------------------------
readseq : 0.181 micros/op; 612.3 MB/s
Signed-off-by: Kyle Zhang <kyle@smartx.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions