jabberwocky238/rocksdb
1
0
Fork 0
mirror of https://github.com/facebook/rocksdb.git synced 2026-07-07 14:47:40 +08:00
Code Issues Packages Projects Releases Wiki Activity

Temp disable in crash test: secondary instance + seqno-time tracking (#13439)

Browse Source 
Summary:
PR https://github.com/facebook/rocksdb/issues/13316 broke some crash test cases in DBImplSecondary, from combining test_secondary=1 and preserve_internal_time_seconds>0. Disabling that while investigating the fix.

Pull Request resolved: https://github.com/facebook/rocksdb/pull/13439

Test Plan: manual blackbox_crash_test runs with forced test_secondary=1

Reviewed By: anand1976

Differential Revision: D70656373

Pulled By: pdillinger

fbshipit-source-id: fa2139e90bbe64ec8ebb062877d9337894ea3b43
This commit is contained in:
Peter Dillinger
2025-03-05 14:32:05 -08:00
committed by Facebook GitHub Bot
parent 15873b1fdd
commit ec8f1452f5
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/db_crashtest.py
+3
View File
@@ -1030,6 +1030,9 @@ def finalize_and_sanitize(src_params):
# Continuous verification fails with secondaries inside NonBatchedOpsStressTest
if dest_params.get("test_secondary") == 1:
dest_params["continuous_verification_interval"] = 0
# FIXME: temporarily broken combination
dest_params["preserve_internal_time_seconds"] = 0
dest_params["preclude_last_level_data_seconds"] = 0
return dest_params