Merge pull request #13513 from vespa-engine/arnej/fix-inconsistent-visibility

keep createFilterSearch public in subclasses

1 files changed, 9 insertions, 10 deletions

diff --git a/searchcore/src/vespa/searchcore/proton/documentmetastore/lid_allocator.cpp b/searchcore/src/vespa/searchcore/proton/documentmetastore/lid_allocator.cpp
index 31361e40c68..ab9072b9449 100644
--- a/searchcore/src/vespa/searchcore/proton/documentmetastore/lid_allocator.cpp
+++ b/searchcore/src/vespa/searchcore/proton/documentmetastore/lid_allocator.cpp
@@ -210,16 +210,6 @@ private:
         (void) tfmda;
         return createFilterSearch(strict, FilterConstraint::UPPER_BOUND);
     }
-
-    SearchIterator::UP createFilterSearch(bool strict, FilterConstraint) const override {
-        auto tfmd = new search::fef::TermFieldMatchData;
-        {
-            std::lock_guard<std::mutex> lock(_lock);
-            _matchDataVector.push_back(tfmd);
-        }
-        return search::BitVectorIterator::create(&_activeLids, get_docid_limit(), *tfmd, strict);
-    }
-
 public:
     WhiteListBlueprint(const search::GrowableBitVector &activeLids)
         : SimpleLeafBlueprint(FieldSpecBaseList()),
@@ -231,6 +221,15 @@ public:
 
     bool isWhiteList() const override { return true; }
 
+    SearchIterator::UP createFilterSearch(bool strict, FilterConstraint) const override {
+        auto tfmd = new search::fef::TermFieldMatchData;
+        {
+            std::lock_guard<std::mutex> lock(_lock);
+            _matchDataVector.push_back(tfmd);
+        }
+        return search::BitVectorIterator::create(&_activeLids, get_docid_limit(), *tfmd, strict);
+    }
+
     ~WhiteListBlueprint() {
         for (auto matchData : _matchDataVector) {
             delete matchData;