Skip to content

Commit ff6d725

Browse files
committed
optimize query parsing to use single field parser when executing on single field
1 parent 2c8c6aa commit ff6d725

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

modules/elasticsearch/src/main/java/org/elasticsearch/index/query/xcontent/FieldQueryParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ public class FieldQueryParser extends AbstractIndexComponent implements XContent
131131
return query;
132132
}
133133

134-
MapperQueryParser queryParser = parseContext.queryParser(qpSettings);
134+
MapperQueryParser queryParser = parseContext.singleQueryParser(qpSettings);
135135

136136
try {
137137
query = queryParser.parse(qpSettings.queryString());

modules/elasticsearch/src/main/java/org/elasticsearch/index/query/xcontent/QueryParseContext.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,12 +104,12 @@ public IndexCache indexCache() {
104104
return indexQueryParser.indexCache;
105105
}
106106

107-
public MapperQueryParser queryParser(QueryParserSettings settings) {
107+
public MapperQueryParser singleQueryParser(QueryParserSettings settings) {
108108
queryParser.reset(settings);
109109
return queryParser;
110110
}
111111

112-
public MultiFieldMapperQueryParser queryParser(MultiFieldQueryParserSettings settings) {
112+
public MultiFieldMapperQueryParser multiQueryParser(MultiFieldQueryParserSettings settings) {
113113
multiFieldQueryParser.reset(settings);
114114
return multiFieldQueryParser;
115115
}

modules/elasticsearch/src/main/java/org/elasticsearch/index/query/xcontent/QueryStringQueryParser.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,13 +177,13 @@ public class QueryStringQueryParser extends AbstractIndexComponent implements XC
177177
if (qpSettings.fields() != null) {
178178
if (qpSettings.fields().size() == 1) {
179179
qpSettings.defaultField(qpSettings.fields().get(0));
180-
queryParser = parseContext.queryParser(qpSettings);
180+
queryParser = parseContext.singleQueryParser(qpSettings);
181181
} else {
182182
qpSettings.defaultField(null); // reset defaultField when using multi query parser
183-
queryParser = parseContext.queryParser(qpSettings);
183+
queryParser = parseContext.multiQueryParser(qpSettings);
184184
}
185185
} else {
186-
queryParser = parseContext.queryParser(qpSettings);
186+
queryParser = parseContext.singleQueryParser(qpSettings);
187187
}
188188

189189

0 commit comments

Comments
 (0)