Uploaded image for project: 'WiredTiger'
  1. WiredTiger
  2. WT-439

evict-empty, second set of changes.

    • Type: Icon: Task Task
    • Resolution: Done
    • WT1.5.0
    • Affects Version/s: None
    • Component/s: None
    • None

      Michael, this change removes the need for the code in your commit 989e2ffbcfe66b096199729cdac7bddc9d8d3aa0 by detecting the empty, instantiated page during the internal page's reconciliation.

      This isn't going to be any faster, really, than your change, but it fixes WT-345 (the fast-delete page leak), and it cleans up internal page reconciliation: the previous code did a lot of testing of the variables set by __rec_child_modify(), and now there's an explicit status passed back. It is still kind of magical, but at least the magic is in one place.

      Would you please do the review, and especially check if you agree with me we can simply discard the backing leaf page in __rec_child_deleted()? That's the fix for WT-345: I'm not sure why I didn't think of it at the time, but I can't think of any reason we'd ever need that leaf page once we decide the backing page isn't visible to any running transaction.

            Assignee:
            michael.cahill@mongodb.com Michael Cahill (Inactive)
            Reporter:
            keith.bostic@mongodb.com Keith Bostic (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: