]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/commit
Fix a bug in db_stress and an incorrect assertion in FilePickerMultiGet (#5301)
authoranand76 <anand76@devvm1373.frc2.facebook.com>
Tue, 14 May 2019 18:54:52 +0000 (11:54 -0700)
committeranand76 <anand76@devvm1373.frc2.facebook.com>
Fri, 24 May 2019 20:22:04 +0000 (13:22 -0700)
commit0d9bfa6e4d98f29e508efedcde811a45a08d82cd
treeb50eb02cd6c04fe1af780f28b9983c8d69dc8d72
parent9feb730c6e4ee637c5a25bad77fb5d954eb241f7
Fix a bug in db_stress and an incorrect assertion in FilePickerMultiGet (#5301)

Summary:
This PR has two fixes for crash test failures -
1. Fix a bug in TestMultiGet() in db_stress that was passing list of key to MultiGet() in the wrong order, thus ensuring that actual values don't match expected values
2. Remove an incorrect assertion in FilePickerMultiGet::GetNextFileInLevelWithKeys() that checks that files in a level are in sorted order. This is not true with MultiGet(), especially if there are duplicate keys and we may have to go back one file for the next key. Furthermore, this assertion makes more sense when a new version is created, rather than at lookup time

Test -
asan_crash and ubsan_crash tests
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5301

Differential Revision: D15337383

Pulled By: anand1976

fbshipit-source-id: 35092cb15bbc1700e5e823cbe07bfa62f1e9e6c6
db/version_set.cc
tools/db_stress.cc