Two small fixes in db_test

Summary:
Two fixes:
(1) WalDir to pick a directory under TmpDir to allow two tests running in parallel without impacting each other
(2) kBlockBasedTableWithWholeKeyHashIndex is disabled by mistake (I assume). Enable it.

Test Plan: ./db_test

Reviewers: yhchiang, ljin

Reviewed By: ljin

Subscribers: nkg-, igor, dhruba, haobo, leveldb

Differential Revision: https://reviews.facebook.net/D19389
main
sdong 10 years ago
parent c4018e771c
commit 6634844dba
  1. 4
      db/db_test.cc

@ -294,8 +294,8 @@ class DBTest {
protected: protected:
// Sequence of option configurations to try // Sequence of option configurations to try
enum OptionConfig { enum OptionConfig {
kBlockBasedTableWithWholeKeyHashIndex,
kDefault, kDefault,
kBlockBasedTableWithWholeKeyHashIndex,
kBlockBasedTableWithPrefixHashIndex, kBlockBasedTableWithPrefixHashIndex,
kPlainTableFirstBytePrefix, kPlainTableFirstBytePrefix,
kPlainTableAllBytesPrefix, kPlainTableAllBytesPrefix,
@ -467,7 +467,7 @@ class DBTest {
options.db_log_dir = test::TmpDir(); options.db_log_dir = test::TmpDir();
break; break;
case kWalDir: case kWalDir:
options.wal_dir = "/tmp/wal"; options.wal_dir = test::TmpDir() + "/wal";
break; break;
case kManifestFileSize: case kManifestFileSize:
options.max_manifest_file_size = 50; // 50 bytes options.max_manifest_file_size = 50; // 50 bytes

Loading…
Cancel
Save