aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTor Egge <Tor.Egge@online.no>2024-01-24 15:46:55 +0100
committerTor Egge <Tor.Egge@online.no>2024-01-24 15:46:55 +0100
commita3bb8e9a17eece892fd99ff531e1c748e967e0e7 (patch)
treeb20ec64dc4963354c95c65cadc73caf94785aae0
parentb63de07159ad78696b1bb82cc99a43b8e5e27713 (diff)
Avoid bad flattening in streaming search.
-rw-r--r--searchlib/src/vespa/searchlib/query/streaming/near_query_node.h2
-rw-r--r--searchlib/src/vespa/searchlib/query/streaming/phrase_query_node.h2
-rw-r--r--searchlib/src/vespa/searchlib/query/streaming/query.h2
-rw-r--r--searchlib/src/vespa/searchlib/query/streaming/same_element_query_node.h2
4 files changed, 4 insertions, 4 deletions
diff --git a/searchlib/src/vespa/searchlib/query/streaming/near_query_node.h b/searchlib/src/vespa/searchlib/query/streaming/near_query_node.h
index b49d17ff936..9258c3efe27 100644
--- a/searchlib/src/vespa/searchlib/query/streaming/near_query_node.h
+++ b/searchlib/src/vespa/searchlib/query/streaming/near_query_node.h
@@ -21,7 +21,7 @@ public:
void distance(size_t dist) { _distance = dist; }
size_t distance() const { return _distance; }
void visitMembers(vespalib::ObjectVisitor &visitor) const override;
- bool isFlattenable(ParseItem::ItemType type) const override { return type == ParseItem::ITEM_NOT; }
+ bool isFlattenable(ParseItem::ItemType) const override { return false; }
private:
size_t _distance;
};
diff --git a/searchlib/src/vespa/searchlib/query/streaming/phrase_query_node.h b/searchlib/src/vespa/searchlib/query/streaming/phrase_query_node.h
index f28684ffeb4..09475178da5 100644
--- a/searchlib/src/vespa/searchlib/query/streaming/phrase_query_node.h
+++ b/searchlib/src/vespa/searchlib/query/streaming/phrase_query_node.h
@@ -20,7 +20,7 @@ public:
void getPhrases(ConstQueryNodeRefList & tl) const override;
const QueryTerm::FieldInfo & getFieldInfo(size_t fid) const { return _fieldInfo[fid]; }
size_t getFieldInfoSize() const { return _fieldInfo.size(); }
- bool isFlattenable(ParseItem::ItemType type) const override { return type == ParseItem::ITEM_NOT; }
+ bool isFlattenable(ParseItem::ItemType) const override { return false; }
void addChild(QueryNode::UP child) override;
private:
mutable std::vector<QueryTerm::FieldInfo> _fieldInfo;
diff --git a/searchlib/src/vespa/searchlib/query/streaming/query.h b/searchlib/src/vespa/searchlib/query/streaming/query.h
index 4ab33a01d86..5296d3a4f69 100644
--- a/searchlib/src/vespa/searchlib/query/streaming/query.h
+++ b/searchlib/src/vespa/searchlib/query/streaming/query.h
@@ -76,7 +76,7 @@ class AndNotQueryNode : public QueryConnector
public:
AndNotQueryNode() noexcept : QueryConnector("ANDNOT") { }
bool evaluate() const override;
- bool isFlattenable(ParseItem::ItemType type) const override { return type == ParseItem::ITEM_NOT; }
+ bool isFlattenable(ParseItem::ItemType) const override { return false; }
};
/**
diff --git a/searchlib/src/vespa/searchlib/query/streaming/same_element_query_node.h b/searchlib/src/vespa/searchlib/query/streaming/same_element_query_node.h
index 50a479eda94..8e675feb569 100644
--- a/searchlib/src/vespa/searchlib/query/streaming/same_element_query_node.h
+++ b/searchlib/src/vespa/searchlib/query/streaming/same_element_query_node.h
@@ -15,7 +15,7 @@ public:
SameElementQueryNode() noexcept : AndQueryNode("SAME_ELEMENT") { }
bool evaluate() const override;
const HitList & evaluateHits(HitList & hl) const override;
- bool isFlattenable(ParseItem::ItemType type) const override { return type == ParseItem::ITEM_NOT; }
+ bool isFlattenable(ParseItem::ItemType) const override { return false; }
void addChild(QueryNode::UP child) override;
};