diff options
author | Keith Bostic <keith@wiredtiger.com> | 2014-08-20 16:13:27 -0400 |
---|---|---|
committer | Keith Bostic <keith@wiredtiger.com> | 2014-08-20 16:13:27 -0400 |
commit | c9e960986052e732ea78b5ed34530c863562ea5c (patch) | |
tree | e0cc885a30a5066caa04bad1176a384070716827 /src/btree/bt_slvg.c | |
parent | a8ce4e3f6006efea12ea7c574e9f1d647a5889bc (diff) | |
download | mongo-c9e960986052e732ea78b5ed34530c863562ea5c.tar.gz |
Trivial comment cleanup.
Diffstat (limited to 'src/btree/bt_slvg.c')
-rw-r--r-- | src/btree/bt_slvg.c | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/src/btree/bt_slvg.c b/src/btree/bt_slvg.c index 0ba5d13a459..75be2b93bd7 100644 --- a/src/btree/bt_slvg.c +++ b/src/btree/bt_slvg.c @@ -1164,19 +1164,20 @@ __slvg_col_build_internal( /* * If the page's key range is unmodified from when we read it * (in other words, we didn't merge part of this page with - * another page), we can use the page without change. If we - * did merge with another page, we must build a page reflecting - * the updated key range, and that requires an additional pass - * to free its backing blocks. + * another page), we can use the page without change, and the + * only thing we need to do is mark all overflow records the + * page references as in-use. + * + * If we did merge with another page, we have to build a page + * reflecting the updated key range. Note, that requires an + * additional pass to free the merge page's backing blocks. */ if (F_ISSET(trk, WT_TRACK_MERGE)) { ss->merge_free = 1; WT_ERR(__slvg_col_build_leaf(session, trk, ref)); - } else { - /* All overflow items referenced by this page used. */ + } else WT_ERR(__slvg_ovfl_ref_all(session, trk)); - } ++ref; } @@ -1775,10 +1776,13 @@ __slvg_row_build_internal( /* * If the page's key range is unmodified from when we read it * (in other words, we didn't merge part of this page with - * another page), we can use the page without change. If we - * did merge with another page, we must build a page reflecting - * the updated key range, and that requires an additional pass - * to free its backing blocks. + * another page), we can use the page without change, and the + * only thing we need to do is mark all overflow records the + * page references as in-use. + * + * If we did merge with another page, we have to build a page + * reflecting the updated key range. Note, that requires an + * additional pass to free the merge page's backing blocks. */ if (F_ISSET(trk, WT_TRACK_MERGE)) { ss->merge_free = 1; @@ -1789,7 +1793,6 @@ __slvg_row_build_internal( trk->row_start.data, trk->row_start.size, &ref->key.ikey)); - /* All overflow items referenced by this page used. */ WT_ERR(__slvg_ovfl_ref_all(session, trk)); } ++ref; |