From f28207d31857d49e3693a5171245ce9d390067ab Mon Sep 17 00:00:00 2001 From: Yuval Ariel Date: Wed, 24 Jan 2024 10:11:08 +0200 Subject: [PATCH] stress test: Disable tracing with use_txn This should be reverted once https://github.com/speedb-io/speedb/issues/813 is handled --- db_stress_tool/db_stress_test_base.cc | 5 +++++ tools/db_crashtest.py | 8 +++++--- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/db_stress_tool/db_stress_test_base.cc b/db_stress_tool/db_stress_test_base.cc index abcdf86ca1..004f05d6a5 100644 --- a/db_stress_tool/db_stress_test_base.cc +++ b/db_stress_tool/db_stress_test_base.cc @@ -2760,6 +2760,11 @@ void StressTest::Open(SharedState* shared) { exit(1); } + if (FLAGS_use_txn && FLAGS_trace_ops) { + fprintf(stderr, "use_txn is currently incompatible with trace_ops.\n"); + exit(1); + } + fprintf(stdout, "Integrated BlobDB: blob files enabled %d, min blob size %" PRIu64 ", blob file size %" PRIu64 diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index 44f56d1dec..71d17eec64 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -821,9 +821,11 @@ def finalize_and_sanitize(src_params, counter): dest_params["unordered_write"] = 0 # For TransactionDB, correctness testing with unsync data loss is currently # compatible with only write committed policy - if dest_params.get("use_txn") == 1 and dest_params.get("txn_write_policy") != 0: - dest_params["sync_fault_injection"] = 0 - dest_params["manual_wal_flush_one_in"] = 0 + if dest_params.get("use_txn") == 1: + dest_params["trace_ops"] = 0 + if dest_params.get("txn_write_policy") != 0: + dest_params["sync_fault_injection"] = 0 + dest_params["manual_wal_flush_one_in"] = 0 # PutEntity is currently not supported by SstFileWriter or in conjunction with Merge if dest_params.get("use_put_entity_one_in", 0) != 0: dest_params["ingest_external_file_one_in"] = 0