timing_stress_rec_before_wrapup never gets selected in test/model

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • WT12.0.0, 8.3.0-rc0
    • Affects Version/s: None
    • Component/s: Test Model
    • None
    • Storage Engines, Storage Engines - Persistence, Storage Engines - Transactions
    • None
    • None

      test/model specifies the following default probabilities for various stress events:

          timing_stress_ckpt_slow = 0.1;
          timing_stress_ckpt_evict_page = 0.1;
          timing_stress_ckpt_handle = 0.1;
          timing_stress_ckpt_stop = 0.1;
          timing_stress_compact_slow = 0.1;
          timing_stress_hs_ckpt_delay = 0.1;
          timing_stress_hs_search = 0.1;
          timing_stress_hs_sweep_race = 0.1;
          timing_stress_prepare_ckpt_delay = 0.1;
          timing_stress_commit_txn_slow = 0.1;
          timing_stress_rec_before_wrapup = 0.1;
      

      Given that these probabilities add up to more than 1.0, the last one in the list, timing_stress_rec_before_wrapup, never gets selected.

      An easy way to fix this would be to update the probabilities in this function and add a comment.

      Another way would be to add up the probabilities in kv_workload_generator::generate_connection_stress_config() and then multiply the random number by the total. However, we must ensure that we don't forget to add one of those factors; ideally we would invent some macro to help with that, so that we would specify the list of timing stress variables only once.

            Assignee:
            Peter Macko
            Reporter:
            Peter Macko
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: