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

correct IndexQueryParserService.parse(QueryBuilder<?>) so that it doesn't convert to JSON and back again

This commit is contained in:
Colin Goodheart-Smithe 2015-09-23 23:58:13 +01:00
parent f54155195a
commit cc9519eee3
1 changed files with 6 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
core/src/main/java/org/elasticsearch/index/query/IndexQueryParserService.java
View File

@ -154,20 +154,14 @@ public class IndexQueryParserService extends AbstractIndexComponent {
}
//norelease this needs to go away
public ParsedQuery parse(QueryBuilder queryBuilder) {
XContentParser parser = null;
public ParsedQuery parse(QueryBuilder<?> queryBuilder) {
QueryShardContext context = cache.get();
try {
BytesReference bytes = queryBuilder.buildAsBytes();
parser = XContentFactory.xContent(bytes).createParser(bytes);
return innerParse(cache.get(), parser);
return innerParse(context, queryBuilder);
} catch (ParsingException e) {
throw e;
} catch (Exception e) {
throw new ParsingException(parser == null ? null : parser.getTokenLocation(), "Failed to parse", e);
} finally {
if (parser != null) {
parser.close();
}
throw new QueryShardException(context, "failed to create query: {}", queryBuilder);
}
}
@ -236,7 +230,7 @@ public class IndexQueryParserService extends AbstractIndexComponent {
}
@Nullable
public QueryBuilder parseInnerQueryBuilder(QueryParseContext parseContext) throws IOException {
public QueryBuilder<?> parseInnerQueryBuilder(QueryParseContext parseContext) throws IOException {
parseContext.parseFieldMatcher(parseFieldMatcher);
return parseContext.parseInnerQueryBuilder();
}
@ -310,7 +304,7 @@ public class IndexQueryParserService extends AbstractIndexComponent {
}
}
private static ParsedQuery innerParse(QueryShardContext context, QueryBuilder queryBuilder) throws IOException, QueryShardException {
private static ParsedQuery innerParse(QueryShardContext context, QueryBuilder<?> queryBuilder) throws IOException, QueryShardException {
Query query = queryBuilder.toQuery(context);
if (query == null) {
query = Queries.newMatchNoDocsQuery();