Renable cookie size assert

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • WT12.0.0, 8.3.0-rc0
    • Affects Version/s: None
    • Component/s: Checkpoints
    • None

      The task compatibility-test-against-develop is/was blocking the merge of the feature branch to develop, this was occurring as an assert we added isn't correct if you run the newer binary on develop.

      To workaround this we are disabling the assert and will re-enable it post merge.

      Patch:

      diff --git a/src/block_disagg/block_disagg_read.c b/src/block_disagg/block_disagg_read.c
      index 0475f80ae3..fe21c2f935 100644
      --- a/src/block_disagg/block_disagg_read.c
      +++ b/src/block_disagg/block_disagg_read.c
      @@ -273,8 +273,7 @@ corrupt:
           }
      
           /* The cumulative size from the cookie must match the sum of all individual block sizes. */
      -    /* FIXME-WT-16667: Re-enable this assertion. */
      -    WT_ASSERT(session, true);
      +    WT_ASSERT(session, block_meta->cumulative_size == block_size_sum);
      
       err:
           time_stop = __wt_clock(session);
      
      

            Assignee:
            Mariam Mojid
            Reporter:
            Luke Pearson
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: