]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/commit
projects / rocksdb.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 1d5083a) | patch
Fix VerifyChecksum readahead with mmap mode (#5945)
authorsdong <siying.d@fb.com>
Mon, 21 Oct 2019 18:37:09 +0000 (11:37 -0700)
committersdong <siying.d@fb.com>
Tue, 22 Oct 2019 18:42:36 +0000 (11:42 -0700)
commitd72cceb443f02e5343b43a26de86676ee7a1d468
tree2d3d678768d0107845ab751173d31605d3f65ddftree | snapshot
parent1d5083a007f5c6ab55fa5fd451da4d2c20975c23commit | diff
Fix VerifyChecksum readahead with mmap mode (#5945)

Summary:
A recent change introduced readahead inside VerifyChecksum(). However it is not compatible with mmap mode and generated wrong checksum verification failure. Fix it by not enabling readahead in mmap
 mode.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5945

Test Plan: Add a unit test that used to fail.

Differential Revision: D18021443

fbshipit-source-id: 6f2eb600f81b26edb02222563a4006869d576bff
db/corruption_test.cc diff | blob | history
table/block_based/block_based_table_reader.cc diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.