Skip to content

Commit

Permalink
stress test: Disable tracing with use_txn
Browse files Browse the repository at this point in the history
This should be reverted once #813 is handled
  • Loading branch information
Yuval-Ariel committed Jan 24, 2024
1 parent 5fd550e commit f28207d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
5 changes: 5 additions & 0 deletions db_stress_tool/db_stress_test_base.cc
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
8 changes: 5 additions & 3 deletions tools/db_crashtest.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit f28207d

Please sign in to comment.