Update TODO comment in test/format related to salvage

XMLWordPrintableJSON

    • Storage Engines, Storage Engines - Foundations, Storage Engines - Persistence
    • SE Persistence - 2025-11-07
    • 1

      The salvage operation has been disabled in test/format through WT-14697.

      An initial attempt at re-enabling it led to WT-15878. Once this is done, we can enable it back inside test/format. Should be as simple as:

      diff --git a/test/format/format_config.c b/test/format/format_config.c
index 4500cb2fa9..176eae4624 100644
--- a/test/format/format_config.c
+++ b/test/format/format_config.c
@@ -494,9 +494,6 @@ config_run(void)
 
     tables_apply(config_table, NULL); /* Configure the tables. */
 
-    /* TODO: Temporarily disable salvage test due to increased failures. */
-    config_off(NULL, "ops.salvage");
-
     /* Order can be important, don't shuffle without careful consideration. */
     config_tiered_storage();                         /* Tiered storage */
     config_disagg_storage();                         /* Disaggregated storage */

      EDIT: Re-purposing the ticket to update the TODO comment to FIXME-WT-12983 since we are still hitting this error when salvage is enabled.

            Assignee:
            Etienne Petrel
            Reporter:
            Etienne Petrel
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: