Skip to content

Commit d4665e1

Browse files
author
Rahul Anishetty
committed
changed to bool must for filtered query builder
1 parent 70f4116 commit d4665e1

File tree

1 file changed

+20
-16
lines changed

1 file changed

+20
-16
lines changed

src/main/java/org/elasticsearch/index/query/FilteredQueryBuilder.java

Lines changed: 20 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -69,25 +69,29 @@ public FilteredQueryBuilder queryName(String queryName) {
6969

7070
@Override
7171
protected void doXContent(XContentBuilder builder, Params params) throws IOException {
72-
builder.startObject(FilteredQueryParser.NAME);
73-
if (queryBuilder != null) {
74-
builder.field("query");
75-
queryBuilder.toXContent(builder, params);
76-
}
77-
if (filterBuilder != null) {
78-
if (ToXContentUtils.getVersionFromParams(params).onOrAfter(Version.V_5_0_0)) {
72+
if (ToXContentUtils.getVersionFromParams(params).onOrAfter(Version.V_5_0_0)) {
73+
if (queryBuilder != null)
74+
QueryBuilders.boolQuery().must(queryBuilder).boost(boost).queryName(queryName).toXContent(builder, params);
75+
if (filterBuilder != null) {
76+
FilterBuilders.boolFilter().must(filterBuilder).toXContent(builder, params);
77+
}
78+
} else {
79+
builder.startObject(FilteredQueryParser.NAME);
80+
if (queryBuilder != null) {
7981
builder.field("query");
80-
} else {
82+
queryBuilder.toXContent(builder, params);
83+
}
84+
if (filterBuilder != null) {
8185
builder.field("filter");
86+
filterBuilder.toXContent(builder, params);
8287
}
83-
filterBuilder.toXContent(builder, params);
84-
}
85-
if (boost != -1) {
86-
builder.field("boost", boost);
87-
}
88-
if (queryName != null) {
89-
builder.field("_name", queryName);
88+
if (boost != -1) {
89+
builder.field("boost", boost);
90+
}
91+
if (queryName != null) {
92+
builder.field("_name", queryName);
93+
}
94+
builder.endObject();
9095
}
91-
builder.endObject();
9296
}
9397
}

0 commit comments

Comments
 (0)