From 82d42606ad2d1f576f25aac0a90c5d854c4f7af3 Mon Sep 17 00:00:00 2001 From: sdong Date: Fri, 9 Oct 2020 08:30:44 -0700 Subject: [PATCH] Enable paranoid_file_checks in crash test (#7489) Summary: Cover paranoid_file_checks in crash test. Pull Request resolved: https://github.com/facebook/rocksdb/pull/7489 Test Plan: Run crash tests for hours and didn't see any failure. Reviewed By: ajkr Differential Revision: D24063868 fbshipit-source-id: 7b48b110e66ce78ae5d0c99a9f32af86edd34c1e --- tools/db_crashtest.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index cd36c663c..4f67c4e89 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -134,8 +134,7 @@ default_params = { "read_fault_one_in": lambda: random.choice([0, 1000]), "sync_fault_injection": False, "get_property_one_in": 1000000, - # paranoid_file_checks has a bug so it's not yet passed. - "paranoid_file_checks": 0, + "paranoid_file_checks": lambda: random.choice([0, 1, 1, 1]), } _TEST_DIR_ENV_VAR = 'TEST_TMPDIR' @@ -200,8 +199,7 @@ simple_default_params = { "test_batches_snapshots": 0, "write_buffer_size": 32 * 1024 * 1024, "level_compaction_dynamic_level_bytes": False, - # "paranoid_file_checks" has a bug so it's not yet passed. - "paranoid_file_checks": 0, + "paranoid_file_checks": lambda: random.choice([0, 1, 1, 1]), } blackbox_simple_default_params = {