From d6b79e2fd03985e59ccb3bf6c67c9e1f80cc91bb Mon Sep 17 00:00:00 2001 From: Islam AbdelRahman Date: Tue, 21 Jun 2016 17:30:21 -0700 Subject: [PATCH] Remove filter_deletes from crash_test Summary: filter_deletes option was removed, remove it from crash_test to fix it Test Plan: make crash_test Reviewers: yhchiang, andrewkr, sdong Reviewed By: sdong Subscribers: andrewkr, dhruba Differential Revision: https://reviews.facebook.net/D59901 --- tools/db_crashtest.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index 51cbcbe6d..a2b0caae8 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -23,7 +23,6 @@ default_params = { "destroy_db_initially": 0, "disable_data_sync": 0, "disable_wal": 0, - "filter_deletes": lambda: random.randint(0, 1), "allow_concurrent_memtable_write": 0, "iterpercent": 10, "max_background_compactions": 20, @@ -87,7 +86,6 @@ simple_default_params = { "destroy_db_initially": 0, "disable_data_sync": 0, "disable_wal": 0, - "filter_deletes": lambda: random.randint(0, 1), "allow_concurrent_memtable_write": lambda: random.randint(0, 1), "iterpercent": 10, "max_background_compactions": 1, @@ -135,9 +133,7 @@ whitebox_simple_default_params = { def finalize_and_sanitize(src_params): dest_params = dict([(k, v() if callable(v) else v) for (k, v) in src_params.items()]) - # --allow_concurrent_memtable_write with --filter_deletes is not supported. if dest_params.get("allow_concurrent_memtable_write", 1) == 1: - dest_params["filter_deletes"] = 0 dest_params["memtablerep"] = "skip_list" return dest_params