Update TODO comment in test/format related to salvage

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • WT12.0.0
    • Affects Version/s: None
    • Component/s: Salvage, Test Format
    • None
    • 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: