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

DBImpl::CheckConsistency() shouldn't create path name with double "/"

Browse Source 
Summary: GetLiveFilesMetaData() already adds a leading "/" in file name. No need to add one extra "/" in DBImpl::CheckConsistency()

Test Plan: make all check

Reviewers: yhchiang, rven, igor

Reviewed By: igor

Subscribers: leveldb, dhruba

Differential Revision: https://reviews.facebook.net/D31779
This commit is contained in:
sdong 2015-01-16 15:44:55 -08:00
parent 5e98e53249
commit 206237d121
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
db/db_impl.cc
View File

@ -3530,7 +3530,8 @@ Status DBImpl::CheckConsistency() {
std::string corruption_messages; std::string corruption_messages;
for (const auto& md : metadata) { for (const auto& md : metadata) {
std::string file_path = md.db_path + "/" + md.name; // md.name has a leading "/".
std::string file_path = md.db_path + md.name;
uint64_t fsize = 0; uint64_t fsize = 0;
Status s = env_->GetFileSize(file_path, &fsize); Status s = env_->GetFileSize(file_path, &fsize);