honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix highlighter support in PinnedQuery and added test (#53716) (#53729) 

CappedScoreQuery was not delegating queryVisitor calls

Closes #53699
This commit is contained in:
markharwood 2020-03-18 15:39:17 +00:00 committed by GitHub
parent df0ad7569b
commit ae19802e29
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 32 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
x-pack/plugin/search-business-rules/src/main/java/org/apache/lucene/search/CappedScoreQuery.java
View File

@ -33,6 +33,11 @@ public final class CappedScoreQuery extends Query {
return query;
}
@Override
public void visit(QueryVisitor visitor) {
query.visit(visitor.getSubVisitor(BooleanClause.Occur.MUST, this));
}
@Override
public Query rewrite(IndexReader reader) throws IOException {
Query rewritten = query.rewrite(reader);

27
x-pack/plugin/search-business-rules/src/test/java/org/elasticsearch/xpack/searchbusinessrules/PinnedQueryBuilderIT.java
View File

@ -17,12 +17,15 @@ import org.elasticsearch.index.query.QueryBuilders;
import org.elasticsearch.index.query.QueryStringQueryBuilder;
import org.elasticsearch.plugins.Plugin;
import org.elasticsearch.search.SearchHit;
import org.elasticsearch.search.fetch.subphase.highlight.HighlightBuilder;
import org.elasticsearch.search.fetch.subphase.highlight.HighlightField;
import org.elasticsearch.test.ESIntegTestCase;
import java.util.ArrayList;
import java.util.Collection;
import java.util.LinkedHashSet;
import java.util.List;
import java.util.Map;
import static org.elasticsearch.action.search.SearchType.DFS_QUERY_THEN_FETCH;
import static org.elasticsearch.common.xcontent.XContentFactory.jsonBuilder;
@ -198,4 +201,28 @@ public class PinnedQueryBuilderIT extends ESIntegTestCase {
}
public void testHighlight() throws Exception {
// Issue raised in https://github.com/elastic/elasticsearch/issues/53699
assertAcked(prepareCreate("test").addMapping("type1",
jsonBuilder().startObject().startObject("type1").startObject("properties").startObject("field1")
.field("analyzer", "whitespace").field("type", "text").endObject().endObject().endObject().endObject()));
ensureGreen();
client().prepareIndex("test", "type1", "1").setSource("field1", "the quick brown fox").get();
refresh();
PinnedQueryBuilder pqb = new PinnedQueryBuilder(QueryBuilders.matchQuery("field1", "the quick brown").operator(Operator.OR), "2");
HighlightBuilder testHighlighter = new HighlightBuilder();
testHighlighter.field("field1");
SearchResponse searchResponse = client().prepareSearch().setSearchType(SearchType.DFS_QUERY_THEN_FETCH).setQuery(pqb)
.highlighter(testHighlighter)
.setExplain(true).get();
assertHitCount(searchResponse, 1);
Map<String, HighlightField> highlights = searchResponse.getHits().getHits()[0].getHighlightFields();
assertThat(highlights.size(), equalTo(1));
HighlightField highlight = highlights.get("field1");
assertThat(highlight.fragments()[0].toString(), equalTo("<em>the</em> <em>quick</em> <em>brown</em> fox"));
}
}