From 486f236d01bed55c77c3bdd066412050d29c6d09 Mon Sep 17 00:00:00 2001
From: Henning Baldersheim <balder@yahoo-inc.com>
Date: Sun, 28 Aug 2022 20:16:23 +0000
Subject: Drop separate doom in phrase search. Rely on proper split/delay
 solving the doom issue at a higher level.

---
 .../queryeval/simple_phrase/simple_phrase_test.cpp | 59 ++--------------------
 1 file changed, 3 insertions(+), 56 deletions(-)

(limited to 'searchlib/src/tests/queryeval')

diff --git a/searchlib/src/tests/queryeval/simple_phrase/simple_phrase_test.cpp b/searchlib/src/tests/queryeval/simple_phrase/simple_phrase_test.cpp
index 4877072eacd..13202a062c7 100644
--- a/searchlib/src/tests/queryeval/simple_phrase/simple_phrase_test.cpp
+++ b/searchlib/src/tests/queryeval/simple_phrase/simple_phrase_test.cpp
@@ -48,9 +48,6 @@ class Test : public vespalib::TestApp {
     void requireThatTermsCanBeEvaluatedInPriorityOrder();
     void requireThatBlueprintExposesFieldWithEstimate();
     void requireThatBlueprintForcesPositionDataOnChildren();
-    void requireThatIteratorHonorsFutureDoom();
-    void requireThatIteratorHonorsDoom();
-    void requireThatDoomIsPropagated();
 
 public:
     int Main() override;
@@ -79,9 +76,6 @@ Test::Main()
     TEST_DO(requireThatPhrasesAreUnpacked(true, false, true));
     TEST_DO(requireThatBlueprintExposesFieldWithEstimate());
     TEST_DO(requireThatBlueprintForcesPositionDataOnChildren());
-    TEST_DO(requireThatIteratorHonorsFutureDoom());
-    TEST_DO(requireThatIteratorHonorsDoom());
-    TEST_DO(requireThatDoomIsPropagated());
 
     TEST_DONE();
 }
@@ -187,7 +181,7 @@ PhraseSearchTest::PhraseSearchTest(bool expiredDoom)
     : _requestContext(nullptr, expiredDoom ? vespalib::steady_time(): vespalib::steady_time::max()),
       _index(),
       _phrase_fs(field, fieldId, phrase_handle),
-      _phrase(_phrase_fs, _requestContext, false),
+      _phrase(_phrase_fs, false),
       _children(),
       _md(MatchData::makeTestInstance(100, 10)),
       _order(),
@@ -207,51 +201,6 @@ void Test::requireThatIteratorFindsSimplePhrase(bool useBlueprint) {
     EXPECT_TRUE(!search->seek(doc_no_match));
 }
 
-void Test::requireThatIteratorHonorsFutureDoom() {
-    PhraseSearchTest test;
-    test.addTerm("foo", 0).addTerm("bar", 1);
-
-    test.fetchPostings(false);
-    vespalib::TestClock clock;
-    vespalib::Doom futureDoom(clock.clock(), vespalib::steady_time::max());
-    unique_ptr<SearchIterator> search(test.createSearch(false));
-    static_cast<SimplePhraseSearch &>(*search).setDoom(&futureDoom);
-    EXPECT_TRUE(!search->seek(1u));
-    EXPECT_TRUE(search->seek(doc_match));
-    EXPECT_TRUE(!search->seek(doc_no_match));
-}
-
-void Test::requireThatIteratorHonorsDoom() {
-    PhraseSearchTest test;
-    test.addTerm("foo", 0).addTerm("bar", 1);
-
-    test.fetchPostings(false);
-    vespalib::TestClock clock;
-    vespalib::Doom futureDoom(clock.clock(), vespalib::steady_time());
-    unique_ptr<SearchIterator> search(test.createSearch(false));
-    static_cast<SimplePhraseSearch &>(*search).setDoom(&futureDoom);
-    EXPECT_TRUE(!search->seek(1u));
-    EXPECT_EQUAL(search->beginId(), search->getDocId());
-    EXPECT_TRUE(!search->seek(doc_match));
-    EXPECT_TRUE(search->isAtEnd());
-    EXPECT_TRUE(!search->seek(doc_no_match));
-    EXPECT_TRUE(search->isAtEnd());
-}
-
-void Test::requireThatDoomIsPropagated() {
-    PhraseSearchTest test(true);
-    test.addTerm("foo", 0).addTerm("bar", 1);
-
-    test.fetchPostings(true);
-    unique_ptr<SearchIterator> search(test.createSearch(true));
-    EXPECT_TRUE(!search->seek(1u));
-    EXPECT_EQUAL(search->beginId(), search->getDocId());
-    EXPECT_TRUE(!search->seek(doc_match));
-    EXPECT_TRUE(search->isAtEnd());
-    EXPECT_TRUE(!search->seek(doc_no_match));
-    EXPECT_TRUE(search->isAtEnd());
-}
-
 void Test::requireThatIteratorFindsLongPhrase(bool useBlueprint) {
     PhraseSearchTest test;
     test.addTerm("foo", 0).addTerm("bar", 0).addTerm("baz", 0)
@@ -326,9 +275,8 @@ void Test::requireThatTermsCanBeEvaluatedInPriorityOrder() {
 void
 Test::requireThatBlueprintExposesFieldWithEstimate()
 {
-    FakeRequestContext requestContext;
     FieldSpec f("foo", 1, 1);
-    SimplePhraseBlueprint phrase(f, requestContext, false);
+    SimplePhraseBlueprint phrase(f, false);
     ASSERT_TRUE(phrase.getState().numFields() == 1);
     EXPECT_EQUAL(f.getFieldId(), phrase.getState().field(0).getFieldId());
     EXPECT_EQUAL(f.getHandle(), phrase.getState().field(0).getHandle());
@@ -352,9 +300,8 @@ Test::requireThatBlueprintExposesFieldWithEstimate()
 void
 Test::requireThatBlueprintForcesPositionDataOnChildren()
 {
-    FakeRequestContext requestContext;
     FieldSpec f("foo", 1, 1, true);
-    SimplePhraseBlueprint phrase(f, requestContext, false);
+    SimplePhraseBlueprint phrase(f, false);
     EXPECT_TRUE(f.isFilter());
     EXPECT_TRUE(!phrase.getNextChildField(f).isFilter());
 }
-- 
cgit v1.2.3