Skip to main content

Browse the archive

Revision cadc1adffa447f02eb65bd848cf26c13142a74bb authored by sdong on 13 June 2014, 22:54:19 UTC, committed by sdong on 16 June 2014, 23:10:52 UTC 
Refactor: group metadata needed to open an SST file to a separate copyable struct
 
Summary:
We added multiple fields to FileMetaData recently and are planning to add more.
This refactoring separate the minimum information for accessing the file. This object is copyable (FileMetaData is not copyable since the ref counter). I hope this refactoring can enable further improvements:

(1) use it to design a more efficient data structure to speed up read queries.
(2) in the future, when we add information of storage level, we can easily do the encoding, instead of enlarge this structure, which might expand memory work set for file meta data.

The definition is same as current EncodedFileMetaData used in two level iterator, so now the logic in two level iterator is easier to understand.

Test Plan: make all check

Reviewers: haobo, igor, ljin

Reviewed By: ljin

Subscribers: leveldb, dhruba, yhchiang

Differential Revision: https://reviews.facebook.net/D18933
1 parent 4d913cf
Raw File
perf_context.cc 
//  Copyright (c) 2013, Facebook, Inc.  All rights reserved.
//  This source code is licensed under the BSD-style license found in the
//  LICENSE file in the root directory of this source tree. An additional grant
//  of patent rights can be found in the PATENTS file in the same directory.
//

#include <sstream>
#include "util/perf_context_imp.h"

namespace rocksdb {

#if defined(NPERF_CONTEXT) || defined(IOS_CROSS_COMPILE)
PerfLevel perf_level = kEnableCount;
// This is a dummy variable since some place references it
PerfContext perf_context;
#else
__thread PerfLevel perf_level = kEnableCount;
__thread PerfContext perf_context;
#endif

void SetPerfLevel(PerfLevel level) {
  perf_level = level;
}

void PerfContext::Reset() {
#if !defined(NPERF_CONTEXT) && !defined(IOS_CROSS_COMPILE)
  user_key_comparison_count = 0;
  block_cache_hit_count = 0;
  block_read_count = 0;
  block_read_byte = 0;
  block_read_time = 0;
  block_checksum_time = 0;
  block_decompress_time = 0;
  internal_key_skipped_count = 0;
  internal_delete_skipped_count = 0;
  write_wal_time = 0;

  get_snapshot_time = 0;
  get_from_memtable_time = 0;
  get_from_memtable_count = 0;
  get_post_process_time = 0;
  get_from_output_files_time = 0;
  seek_child_seek_time = 0;
  seek_child_seek_count = 0;
  seek_min_heap_time = 0;
  seek_internal_seek_time = 0;
  find_next_user_entry_time = 0;
  write_pre_and_post_process_time = 0;
  write_memtable_time = 0;
#endif
}

#define OUTPUT(counter) #counter << " = " << counter << ", "

std::string PerfContext::ToString() const {
#if defined(NPERF_CONTEXT) || defined(IOS_CROSS_COMPILE)
  return "";
#else
  std::ostringstream ss;
  ss << OUTPUT(user_key_comparison_count)
     << OUTPUT(block_cache_hit_count)
     << OUTPUT(block_read_count)
     << OUTPUT(block_read_byte)
     << OUTPUT(block_read_time)
     << OUTPUT(block_checksum_time)
     << OUTPUT(block_decompress_time)
     << OUTPUT(internal_key_skipped_count)
     << OUTPUT(internal_delete_skipped_count)
     << OUTPUT(write_wal_time)
     << OUTPUT(get_snapshot_time)
     << OUTPUT(get_from_memtable_time)
     << OUTPUT(get_from_memtable_count)
     << OUTPUT(get_post_process_time)
     << OUTPUT(get_from_output_files_time)
     << OUTPUT(seek_child_seek_time)
     << OUTPUT(seek_child_seek_count)
     << OUTPUT(seek_min_heap_time)
     << OUTPUT(seek_internal_seek_time)
     << OUTPUT(find_next_user_entry_time)
     << OUTPUT(write_pre_and_post_process_time)
     << OUTPUT(write_memtable_time);
  return ss.str();
#endif
}

}
back to top