Revision 885b1c682e85391beb7f26a842e2128e3653bd4a authored by Siying Dong on 14 September 2017, 22:41:19 UTC, committed by Facebook Github Bot on 14 September 2017, 22:41:49 UTC
Summary:
Move uncommon code paths in RangeDelAggregator::ShouldDelete() and IterKey::EnlargeBufferIfNeeded() to a separate function, so that the inlined strcuture can be more optimized.

Optimize it because these places show up in CPU profiling, though minimum. The performance is really hard measure. I ran db_bench with readseq benchmark against in-memory DB many times. The variation is big, but it seems to show 1% improvements.
Closes https://github.com/facebook/rocksdb/pull/2877

Differential Revision: D5828123

Pulled By: siying

fbshipit-source-id: 41a49e229f91e9f8409f85cc6f0dc70e31334e4b
1 parent ffac683
History
File Mode Size
win
README -rw-r--r-- 405 bytes
dirent.h -rw-r--r-- 1.1 KB
likely.h -rw-r--r-- 750 bytes
port.h -rw-r--r-- 834 bytes
port_example.h -rw-r--r-- 3.6 KB
port_posix.cc -rw-r--r-- 5.5 KB
port_posix.h -rw-r--r-- 5.6 KB
stack_trace.cc -rw-r--r-- 3.5 KB
stack_trace.h -rw-r--r-- 648 bytes
sys_time.h -rw-r--r-- 1.2 KB
util_logger.h -rw-r--r-- 942 bytes
xpress.h -rw-r--r-- 687 bytes

README

back to top