Skip to content

Commit

Permalink
Don't skip last level when calculating compaction stats
Browse files Browse the repository at this point in the history
Summary: We have a bug where we don't report the last level's files as being compacted. This fixes it.

Test Plan: See the fix in action here: https://phabricator.fb.com/P19845738

Reviewers: MarkCallaghan, sdong

Reviewed By: sdong

Subscribers: yhchiang, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D38727
  • Loading branch information
igorcanadi committed May 22, 2015
1 parent 5c224d1 commit ea6d3a8
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions db/internal_stats.cc
Original file line number Diff line number Diff line change
Expand Up @@ -541,8 +541,7 @@ void InternalStats::DumpCFStats(std::string* value) {
const VersionStorageInfo* vstorage = cfd_->current()->storage_info();

int num_levels_to_check =
(cfd_->ioptions()->compaction_style != kCompactionStyleUniversal &&
cfd_->ioptions()->compaction_style != kCompactionStyleFIFO)
(cfd_->ioptions()->compaction_style != kCompactionStyleFIFO)
? vstorage->num_levels() - 1
: 1;

Expand All @@ -555,7 +554,7 @@ void InternalStats::DumpCFStats(std::string* value) {
}
// Count # of files being compacted for each level
std::vector<int> files_being_compacted(number_levels_, 0);
for (int level = 0; level < num_levels_to_check; ++level) {
for (int level = 0; level < number_levels_; ++level) {
for (auto* f : vstorage->LevelFiles(level)) {
if (f->being_compacted) {
++files_being_compacted[level];
Expand Down

0 comments on commit ea6d3a8

Please sign in to comment.