From: Feng Zhu <zagfox@fb.com>
Date: Mon, 11 Aug 2014 17:47:52 +0000 (-0700)
Subject: check prefix_size when using hash search in db_bench
X-Git-Tag: v3.4~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=d3f2ec694f4fb6c7385a6eea943f6844244dd5c2;p=rocksdb.git

check prefix_size when using hash search in db_bench

Summary:
1. Check prefix_size when enable use_hash_search in db_bench
2. Remove include/statistics.h in db_bench

Test Plan: ./db_bench --use_hash_search=1

Reviewers: ljin, yhchiang, igor, sdong

Reviewed By: sdong

Subscribers: leveldb

Differential Revision: https://reviews.facebook.net/D21375
---

diff --git a/db/db_bench.cc b/db/db_bench.cc
index d4811cb4..41e707f7 100644
--- a/db/db_bench.cc
+++ b/db/db_bench.cc
@@ -30,7 +30,6 @@ int main() {
 #include <gflags/gflags.h>
 #include "db/db_impl.h"
 #include "db/version_set.h"
-#include "rocksdb/statistics.h"
 #include "rocksdb/options.h"
 #include "rocksdb/cache.h"
 #include "rocksdb/db.h"
@@ -1703,6 +1702,11 @@ class Benchmark {
     } else {
       BlockBasedTableOptions block_based_options;
       if (FLAGS_use_hash_search) {
+        if (FLAGS_prefix_size == 0) {
+          fprintf(stderr,
+              "prefix_size not assigned when enable use_hash_search \n");
+          exit(1);
+        }
         block_based_options.index_type = BlockBasedTableOptions::kHashSearch;
       } else {
         block_based_options.index_type = BlockBasedTableOptions::kBinarySearch;