Skip to content

Commit

Permalink
crashtest: dont reroll skip_list or HashSpdRepFactory (#452)
Browse files Browse the repository at this point in the history
Theres a sanitize check in case of allow_concurrent_memtable_write == 1 which
verifies that the memtablerep supports it. This commit prevents the code
from rerolling if the memtablerep was originally skip_list or
speedb.HashSpdRepFactory
  • Loading branch information
Yuval-Ariel committed May 4, 2023
1 parent 23e8586 commit 79c1f87
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/db_crashtest.py
Original file line number Diff line number Diff line change
Expand Up @@ -680,7 +680,7 @@ def finalize_and_sanitize(src_params, counter):
dest_params["txn_write_policy"] = 1
dest_params["allow_concurrent_memtable_write"] = 1
if dest_params.get("allow_concurrent_memtable_write", 0) == 1:
if (dest_params.get("memtablerep") != "skip_list" or
if (dest_params.get("memtablerep") != "skip_list" and
dest_params.get("memtablerep") != "speedb.HashSpdRepFactory"):
dest_params["memtablerep"] = random.choice(
["skip_list", "speedb.HashSpdRepFactory"]
Expand Down

0 comments on commit 79c1f87

Please sign in to comment.