From 71874c5aaf3b492eff372a71bb43b65bdabe7e55 Mon Sep 17 00:00:00 2001 From: sdong Date: Wed, 29 Jan 2020 16:42:15 -0800 Subject: [PATCH] Fix LITE build with DBTest2.AutoPrefixMode1 (#6346) Summary: DBTest2.AutoPrefixMode1 doesn't pass because auto prefix mode is not supported there. Fix it by disabling the test. Pull Request resolved: https://github.com/facebook/rocksdb/pull/6346 Test Plan: Run DBTest2.AutoPrefixMode1 in lite mode Differential Revision: D19627486 fbshipit-source-id: fbde75260aeecb7e6fc406e09c19a71a95aa5f08 --- db/db_test2.cc | 2 ++ include/rocksdb/options.h | 2 ++ 2 files changed, 4 insertions(+) diff --git a/db/db_test2.cc b/db/db_test2.cc index d9b2250d4a..68de0381b3 100644 --- a/db/db_test2.cc +++ b/db/db_test2.cc @@ -4404,6 +4404,7 @@ TEST_F(DBTest2, BlockBasedTablePrefixGetIndexNotFound) { ASSERT_EQ("ok", Get("b1")); } +#ifndef ROCKSDB_LITE TEST_F(DBTest2, AutoPrefixMode1) { // create a DB with block prefix index BlockBasedTableOptions table_options; @@ -4510,6 +4511,7 @@ TEST_F(DBTest2, AutoPrefixMode1) { ASSERT_EQ("a1", iterator->key().ToString()); } } +#endif // ROCKSDB_LITE } // namespace rocksdb #ifdef ROCKSDB_UNITTESTS_WITH_CUSTOM_OBJECTS_FROM_STATIC_LIBS diff --git a/include/rocksdb/options.h b/include/rocksdb/options.h index 13ebb382bd..e0c7f2f941 100644 --- a/include/rocksdb/options.h +++ b/include/rocksdb/options.h @@ -1254,6 +1254,8 @@ struct ReadOptions { // When true, by default use total_order_seek = true, and RocksDB can // selectively enable prefix seek mode if won't generate a different result // from total_order_seek, based on seek key, and iterator upper bound. + // Not suppported in ROCKSDB_LITE mode, in the way that even with value true + // prefix mode is not used. bool auto_prefix_mode; // Enforce that the iterator only iterates over the same prefix as the seek.