From d35ffd569c20cf721d58c7ce1e3a73f31a322c32 Mon Sep 17 00:00:00 2001 From: Levi Tamasi Date: Thu, 12 Sep 2019 12:10:05 -0700 Subject: [PATCH] Temporarily disable hash index in stress tests (#5792) Summary: PR https://github.com/facebook/rocksdb/issues/4020 implicitly enabled the hash index as well in stress/crash tests, resulting in assertion failures in Block. This patch disables the hash index until we can pinpoint the root cause of these issues. Pull Request resolved: https://github.com/facebook/rocksdb/pull/5792 Test Plan: Ran tools/db_crashtest.py and made sure it only uses index types 0 and 2 (binary search and partitioned index). Differential Revision: D17346777 Pulled By: ltamasi fbshipit-source-id: b4318f37f1fda3ee1bbff4ef2c2f556ca9e6b551 --- tools/db_crashtest.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index 697cd265d..4a393b535 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -41,7 +41,8 @@ default_params = { "enable_pipelined_write": 0, "expected_values_path": expected_values_file.name, "flush_one_in": 1000000, - "index_type": lambda: random.randint(0, 2), + # Temporarily disable hash index + "index_type": lambda: random.choice([0, 2]), "max_background_compactions": 20, "max_bytes_for_level_base": 10485760, "max_key": 100000000,