From c508e8bdf9e2d200388ff86d9dac3818a22f6034 Mon Sep 17 00:00:00 2001 From: Stephen Atherton Date: Tue, 4 Jul 2017 23:49:18 -0700 Subject: [PATCH] Adjacent internal btree pages can now contain the same keys at different versions. --- fdbserver/VersionedBTree.actor.cpp | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/fdbserver/VersionedBTree.actor.cpp b/fdbserver/VersionedBTree.actor.cpp index 6661633046..b4cbff08b8 100755 --- a/fdbserver/VersionedBTree.actor.cpp +++ b/fdbserver/VersionedBTree.actor.cpp @@ -280,17 +280,6 @@ private: results.push_back( {0, {{lowerBoundKey, root}}} ); } - // Verify that no consecutive split keys are equal - // TODO: Is this still needed or do keys already have version suffixes? - StringRef lastSplitKey(LiteralStringRef("\xff\xff\xff")); - for(auto const &p : pages) { - if(p.first != 0) { - StringRef splitKey = merged[p.first].key; - ASSERT(splitKey != lastSplitKey); - lastSplitKey = splitKey; - } - } - // For each IPage of data, assign a logical pageID. std::vector logicalPages;