From: Peter Dillinger Date: Tue, 16 Dec 2025 20:36:07 +0000 (-0800) Subject: Disable multiscan+timestamp in crash test (#14189) X-Git-Tag: v10.10.1~2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=9065ace05ada92e0a435fbb973ef957fc4401f43;p=rocksdb.git Disable multiscan+timestamp in crash test (#14189) Summary: Causing failures and not yet supported. Also putting a note in db.h about the combination being unsupported. Pull Request resolved: https://github.com/facebook/rocksdb/pull/14189 Test Plan: started up blackbox_crash_test_with_ts many times and checked command line to be confident it's excluded. Reviewed By: hx235 Differential Revision: D89297971 Pulled By: pdillinger fbshipit-source-id: c5134351d9ecb37879c7e3319c17dd9228d7f12a --- diff --git a/include/rocksdb/db.h b/include/rocksdb/db.h index 2ac8aa99c..9a753ae3e 100644 --- a/include/rocksdb/db.h +++ b/include/rocksdb/db.h @@ -1105,9 +1105,12 @@ class DB { // details. For optimal performance, ensure that either all entries in // scan_opts specify the range limit, or none of them do. // - // NOTE: iterate_upper_bound in ReadOptions will be ignored. Instead, the - // range.limit in ScanOptions is consulted to determine the upper bound key, - // if specified. + // NOTE: NOT YET SUPPORTED in DBs using user timestamp (see + // Comparator::timestamp_size()) + // + // NOTE: iterate_upper_bound in ReadOptions will + // be ignored. Instead, the range.limit in ScanOptions is consulted to + // determine the upper bound key, if specified. // // Example usage - // std::vector scans{{.start = Slice("bar")}, diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index 160667940..621795ceb 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -692,6 +692,8 @@ ts_params = { # Below flag is randomly picked once and kept consistent in following runs. "persist_user_defined_timestamps": random.choice([0, 1, 1]), "use_merge": 0, + # Causing failures and not yet compatible + "use_multiscan": 0, "use_full_merge_v1": 0, "use_txn": 0, "ingest_external_file_one_in": 0,