diff options
author | Arnstein Ressem <aressem@gmail.com> | 2019-01-07 21:24:20 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-07 21:24:20 +0100 |
commit | 0d69f9fa258df33f00b708f1ee9af58a8f33917c (patch) | |
tree | abfc76398a10cebc8ba584c98a073a25e83eafab /searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp | |
parent | 8191ade44977f2874d60fc6811bbb9427a60510d (diff) |
Revert "Compact lid space on source selector."
Diffstat (limited to 'searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp')
-rw-r--r-- | searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp b/searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp index 0f9df1fc594..c11bea14863 100644 --- a/searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp +++ b/searchcore/src/tests/proton/documentdb/feedview/feedview_test.cpp @@ -140,14 +140,12 @@ struct MyIndexWriter : public test::MockIndexWriter MyLidVector _removes; int _heartBeatCount; uint32_t _commitCount; - uint32_t _wantedLidLimit; MyTracer &_tracer; MyIndexWriter(MyTracer &tracer) : test::MockIndexWriter(IIndexManager::SP(new test::MockIndexManager())), _removes(), _heartBeatCount(0), _commitCount(0), - _wantedLidLimit(0), _tracer(tracer) {} virtual void put(SerialNum serialNum, const document::Document &doc, @@ -166,9 +164,6 @@ struct MyIndexWriter : public test::MockIndexWriter _tracer.traceCommit(indexAdapterTypeName, serialNum); } virtual void heartBeat(SerialNum) override { ++_heartBeatCount; } - void compactLidSpace(SerialNum, uint32_t lidLimit) override { - _wantedLidLimit = lidLimit; - } }; struct MyGidToLidChangeHandler : public MockGidToLidChangeHandler @@ -1149,7 +1144,7 @@ TEST_F("require that compactLidSpace() propagates to document meta store and doc f.compactLidSpaceAndWait(2); // performIndexForceCommit in index thread, then completion callback // in master thread. - EXPECT_TRUE(assertThreadObserver(7, 5, 3, f.writeServiceObserver())); + EXPECT_TRUE(assertThreadObserver(7, 4, 3, f.writeServiceObserver())); EXPECT_EQUAL(2u, f.metaStoreObserver()._compactLidSpaceLidLimit); EXPECT_EQUAL(2u, f.getDocumentStore()._compactLidSpaceLidLimit); EXPECT_EQUAL(1u, f.metaStoreObserver()._holdUnblockShrinkLidSpaceCnt); @@ -1167,7 +1162,7 @@ TEST_F("require that compactLidSpace() doesn't propagate to " op.setSerialNum(0); f.runInMaster([&] () { f.fv.handleCompactLidSpace(op); }); // Delayed holdUnblockShrinkLidSpace() in index thread, then master thread - EXPECT_TRUE(assertThreadObserver(6, 4, 3, f.writeServiceObserver())); + EXPECT_TRUE(assertThreadObserver(6, 3, 3, f.writeServiceObserver())); EXPECT_EQUAL(0u, f.metaStoreObserver()._compactLidSpaceLidLimit); EXPECT_EQUAL(0u, f.getDocumentStore()._compactLidSpaceLidLimit); EXPECT_EQUAL(0u, f.metaStoreObserver()._holdUnblockShrinkLidSpaceCnt); @@ -1181,13 +1176,6 @@ TEST_F("require that compactLidSpace() propagates to attributeadapter", EXPECT_EQUAL(2u, f.maw._wantedLidLimit); } -TEST_F("require that compactLidSpace() propagates to index writer", - SearchableFeedViewFixture) -{ - f.populateBeforeCompactLidSpace(); - f.compactLidSpaceAndWait(2); - EXPECT_EQUAL(2u, f.miw._wantedLidLimit); -} TEST_F("require that commit is called if visibility delay is 0", SearchableFeedViewFixture) |