diff --git a/src/bees-roots.cc b/src/bees-roots.cc index 9661243..be0ba92 100644 --- a/src/bees-roots.cc +++ b/src/bees-roots.cc @@ -742,13 +742,24 @@ BeesCrawl::fetch_extents() if (gen < get_state().m_min_transid) { BEESCOUNT(crawl_gen_low); ++count_low; - // We probably want (need?) to scan these anyway. - // continue; + // We want (need?) to scan these anyway? + // The header generation refers to the transid + // of the metadata page holding the current ref. + // This includes anything else in that page that + // happened to be modified, regardless of how + // old it is. + // The file_extent_generation refers to the + // transid of the extent item's page, which is + // a different approximation of what we want. + // Combine both of these filters to minimize + // the number of times we unnecessarily re-read + // an extent. + continue; } if (gen > get_state().m_max_transid) { BEESCOUNT(crawl_gen_high); ++count_high; - // This shouldn't ever happen + // This shouldn't ever happen...and so far, doesn't. // continue; }