Merge pull request #799 from wiredtiger/overflow-instantiate

Remove call to instantiate a deleted overflow key during reconciliation.

1 files changed, 12 insertions, 9 deletions

diff --git a/src/btree/rec_write.c b/src/btree/rec_write.c
index 6fd89ca9429..93f50ced337 100644
--- a/src/btree/rec_write.c
+++ b/src/btree/rec_write.c
@@ -3454,18 +3454,21 @@ __rec_row_leaf(WT_SESSION_IMPL *session,
 			if (WT_UPDATE_DELETED_ISSET(upd)) {
 				/*
 				 * Overflow keys referencing discarded values
-				 * are no longer useful, schedule the discard
-				 * of the backing blocks.  Don't worry about
-				 * reuse, reusing the key in this reconciliation
-				 * is unlikely.
-				 *
-				 * Keys are part of the name-space though, we
-				 * can't remove them from the in-memory tree;
-				 * if an overflow key was never instantiated,
-				 * do it now.
+				 * are no longer useful, discard the backing
+				 * blocks.  Don't worry about reuse, reusing
+				 * keys from a row-store page reconciliation
+				 * seems unlikely enough to ignore.
 				 */
 				__wt_cell_unpack(cell, unpack);
 				if (unpack->ovfl) {
+					/*
+					 * Keys are part of the name-space, we
+					 * can't remove them from the in-memory
+					 * tree; if an overflow key was deleted
+					 * without being instantiated (for
+					 * example, cursor-based trunction, do
+					 * it now.
+					 */
 					if (ikey == NULL)
 						WT_ERR(__wt_row_leaf_key_work(
 						    session,