diff options
| author | Luca Cavanna <javanna@users.noreply.github.com> | 2016-04-05 15:38:53 +0200 |
|---|---|---|
| committer | Luca Cavanna <javanna@users.noreply.github.com> | 2016-04-05 15:38:53 +0200 |
| commit | 67ccfc354e8e665144f5ac68cf8b3b1257fee619 (patch) | |
| tree | 75abc9adb8829768d6e64d37255f1aab83400bfa /core/src/main/java/org/elasticsearch/index | |
| parent | 84eacadd5130740759fb508de3f1e8ab6cba1448 (diff) | |
Switch to using ParseField to parse query names
* [TEST] check registered queries one by one in SearchModuleTests
* Switch to using ParseField to parse query names
If we have a deprecated query name, at the moment we don't have a way to log any deprecation warning nor fail when we are in strict mode. With this change we use ParseField, which will take care of the camel casing that we currently do manually (so that one day we can remove it more easily). This also means, that each query will have a unique preferred name, and all the other names are deprecated.
Terms query "in" synonym is now formally deprecated, as well as fuzzy_match, match_fuzzy, match_phrase and match_phrase_prefix for match query, mlt for more_like_this and geo_bbox for geo_bounding_box. All these will be removed in 6.0.
Every QueryParser holds now a ParseField constant called QUERY_NAME_FIELD that holds the name for it. The first name is the preferred one, all the others are deprecated. The first name is taken from the NAME constant already present in each query builder object, so that we somehow keep the serialization constant separated from ParseField. This change also allowed us to remove the names method from the QueryParser interface.
Diffstat (limited to 'core/src/main/java/org/elasticsearch/index')
67 files changed, 267 insertions, 386 deletions
diff --git a/core/src/main/java/org/elasticsearch/index/query/BoolQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/BoolQueryBuilder.java index d9c277653e..1a788db738 100644 --- a/core/src/main/java/org/elasticsearch/index/query/BoolQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/BoolQueryBuilder.java @@ -245,7 +245,8 @@ public class BoolQueryBuilder extends AbstractQueryBuilder<BoolQueryBuilder> { builder.endObject(); } - private static void doXArrayContent(String field, List<QueryBuilder<?>> clauses, XContentBuilder builder, Params params) throws IOException { + private static void doXArrayContent(String field, List<QueryBuilder<?>> clauses, XContentBuilder builder, Params params) + throws IOException { if (clauses.isEmpty()) { return; } @@ -284,7 +285,8 @@ public class BoolQueryBuilder extends AbstractQueryBuilder<BoolQueryBuilder> { return adjustPureNegative ? fixNegativeQueryIfNeeded(query) : query; } - private static void addBooleanClauses(QueryShardContext context, BooleanQuery.Builder booleanQueryBuilder, List<QueryBuilder<?>> clauses, Occur occurs) throws IOException { + private static void addBooleanClauses(QueryShardContext context, BooleanQuery.Builder booleanQueryBuilder, + List<QueryBuilder<?>> clauses, Occur occurs) throws IOException { for (QueryBuilder<?> query : clauses) { Query luceneQuery = null; switch (occurs) { @@ -373,7 +375,8 @@ public class BoolQueryBuilder extends AbstractQueryBuilder<BoolQueryBuilder> { return this; } - private static boolean rewriteClauses(QueryRewriteContext queryRewriteContext, List<QueryBuilder<?>> builders, Consumer<QueryBuilder<?>> consumer) throws IOException { + private static boolean rewriteClauses(QueryRewriteContext queryRewriteContext, List<QueryBuilder<?>> builders, + Consumer<QueryBuilder<?>> consumer) throws IOException { boolean changed = false; for (QueryBuilder builder : builders) { QueryBuilder result = builder.rewrite(queryRewriteContext); diff --git a/core/src/main/java/org/elasticsearch/index/query/BoolQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/BoolQueryParser.java index 53c1b81989..78bcabc412 100644 --- a/core/src/main/java/org/elasticsearch/index/query/BoolQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/BoolQueryParser.java @@ -19,19 +19,20 @@ package org.elasticsearch.index.query; -import java.io.IOException; -import java.util.ArrayList; -import java.util.List; - import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; import org.elasticsearch.common.xcontent.XContentParser; +import java.io.IOException; +import java.util.ArrayList; +import java.util.List; + /** * Parser for bool query */ public class BoolQueryParser implements QueryParser<BoolQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(BoolQueryBuilder.NAME); public static final String MUSTNOT = "mustNot"; public static final String MUST_NOT = "must_not"; public static final String FILTER = "filter"; @@ -43,11 +44,6 @@ public class BoolQueryParser implements QueryParser<BoolQueryBuilder> { public static final ParseField ADJUST_PURE_NEGATIVE = new ParseField("adjust_pure_negative"); @Override - public String[] names() { - return new String[]{BoolQueryBuilder.NAME}; - } - - @Override public BoolQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException, ParsingException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/BoostingQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/BoostingQueryParser.java index 76fac798cb..12f7e1e4b0 100644 --- a/core/src/main/java/org/elasticsearch/index/query/BoostingQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/BoostingQueryParser.java @@ -30,16 +30,12 @@ import java.io.IOException; */ public class BoostingQueryParser implements QueryParser<BoostingQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(BoostingQueryBuilder.NAME); public static final ParseField POSITIVE_FIELD = new ParseField("positive"); public static final ParseField NEGATIVE_FIELD = new ParseField("negative"); public static final ParseField NEGATIVE_BOOST_FIELD = new ParseField("negative_boost"); @Override - public String[] names() { - return new String[]{BoostingQueryBuilder.NAME}; - } - - @Override public BoostingQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -86,7 +82,8 @@ public class BoostingQueryParser implements QueryParser<BoostingQueryBuilder> { throw new ParsingException(parser.getTokenLocation(), "[boosting] query requires 'negative' query to be set'"); } if (negativeBoost < 0) { - throw new ParsingException(parser.getTokenLocation(), "[boosting] query requires 'negative_boost' to be set to be a positive value'"); + throw new ParsingException(parser.getTokenLocation(), + "[boosting] query requires 'negative_boost' to be set to be a positive value'"); } BoostingQueryBuilder boostingQuery = new BoostingQueryBuilder(positiveQuery, negativeQuery); diff --git a/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryBuilder.java index a0fc9ee7e7..4efb9d3210 100644 --- a/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryBuilder.java @@ -257,7 +257,8 @@ public class CommonTermsQueryBuilder extends AbstractQueryBuilder<CommonTermsQue Occur highFreqOccur = highFreqOperator.toBooleanClauseOccur(); Occur lowFreqOccur = lowFreqOperator.toBooleanClauseOccur(); - ExtendedCommonTermsQuery commonsQuery = new ExtendedCommonTermsQuery(highFreqOccur, lowFreqOccur, cutoffFrequency, disableCoord, fieldType); + ExtendedCommonTermsQuery commonsQuery = new ExtendedCommonTermsQuery(highFreqOccur, lowFreqOccur, + cutoffFrequency, disableCoord, fieldType); return parseQueryString(commonsQuery, text, field, analyzerObj, lowFreqMinimumShouldMatch, highFreqMinimumShouldMatch); } diff --git a/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryParser.java index c004e1d6c0..8e66835f3b 100644 --- a/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/CommonTermsQueryParser.java @@ -30,6 +30,7 @@ import java.io.IOException; */ public class CommonTermsQueryParser implements QueryParser<CommonTermsQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(CommonTermsQueryBuilder.NAME); public static final ParseField CUTOFF_FREQUENCY_FIELD = new ParseField("cutoff_frequency"); public static final ParseField MINIMUM_SHOULD_MATCH_FIELD = new ParseField("minimum_should_match"); public static final ParseField LOW_FREQ_OPERATOR_FIELD = new ParseField("low_freq_operator"); @@ -41,11 +42,6 @@ public class CommonTermsQueryParser implements QueryParser<CommonTermsQueryBuild public static final ParseField LOW_FREQ_FIELD = new ParseField("low_freq"); @Override - public String[] names() { - return new String[] { CommonTermsQueryBuilder.NAME }; - } - - @Override public CommonTermsQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); XContentParser.Token token = parser.nextToken(); @@ -81,16 +77,19 @@ public class CommonTermsQueryParser implements QueryParser<CommonTermsQueryBuild } else if (parseContext.parseFieldMatcher().match(innerFieldName, HIGH_FREQ_FIELD)) { highFreqMinimumShouldMatch = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + "] query does not support [" + innerFieldName + throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + + "] query does not support [" + innerFieldName + "] for [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + "] unexpected token type [" + token + throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + + "] unexpected token type [" + token + "] after [" + innerFieldName + "]"); } } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else if (token.isValue()) { if (parseContext.parseFieldMatcher().match(currentFieldName, QUERY_FIELD)) { @@ -112,7 +111,8 @@ public class CommonTermsQueryParser implements QueryParser<CommonTermsQueryBuild } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + CommonTermsQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } } @@ -123,7 +123,8 @@ public class CommonTermsQueryParser implements QueryParser<CommonTermsQueryBuild token = parser.nextToken(); if (token != XContentParser.Token.END_OBJECT) { throw new ParsingException(parser.getTokenLocation(), - "[common] query parsed in simplified form, with direct field name, but included more options than just the field name, possibly use its 'options' form, with 'query' element?"); + "[common] query parsed in simplified form, with direct field name, but included more options than just " + + "the field name, possibly use its 'options' form, with 'query' element?"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/ConstantScoreQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/ConstantScoreQueryParser.java index 318a0b3380..cd36a110ef 100644 --- a/core/src/main/java/org/elasticsearch/index/query/ConstantScoreQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/ConstantScoreQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,14 +30,10 @@ import java.io.IOException; */ public class ConstantScoreQueryParser implements QueryParser<ConstantScoreQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(ConstantScoreQueryBuilder.NAME); public static final ParseField INNER_QUERY_FIELD = new ParseField("filter", "query"); @Override - public String[] names() { - return new String[]{ConstantScoreQueryBuilder.NAME, Strings.toCamelCase(ConstantScoreQueryBuilder.NAME)}; - } - - @Override public ConstantScoreQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -63,7 +58,8 @@ public class ConstantScoreQueryParser implements QueryParser<ConstantScoreQueryB query = parseContext.parseInnerQueryBuilder(); queryFound = true; } else { - throw new ParsingException(parser.getTokenLocation(), "[constant_score] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[constant_score] query does not support [" + currentFieldName + "]"); } } else if (token.isValue()) { if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { @@ -71,7 +67,8 @@ public class ConstantScoreQueryParser implements QueryParser<ConstantScoreQueryB } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); } else { - throw new ParsingException(parser.getTokenLocation(), "[constant_score] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[constant_score] query does not support [" + currentFieldName + "]"); } } else { throw new ParsingException(parser.getTokenLocation(), "unexpected token [" + token + "]"); diff --git a/core/src/main/java/org/elasticsearch/index/query/DisMaxQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/DisMaxQueryParser.java index 660790e21a..1ced59d4fd 100644 --- a/core/src/main/java/org/elasticsearch/index/query/DisMaxQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/DisMaxQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -33,15 +32,11 @@ import java.util.List; */ public class DisMaxQueryParser implements QueryParser<DisMaxQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(DisMaxQueryBuilder.NAME); public static final ParseField TIE_BREAKER_FIELD = new ParseField("tie_breaker"); public static final ParseField QUERIES_FIELD = new ParseField("queries"); @Override - public String[] names() { - return new String[]{DisMaxQueryBuilder.NAME, Strings.toCamelCase(DisMaxQueryBuilder.NAME)}; - } - - @Override public DisMaxQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/ExistsQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/ExistsQueryBuilder.java index 79e38fe7be..431e12d004 100644 --- a/core/src/main/java/org/elasticsearch/index/query/ExistsQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/ExistsQueryBuilder.java @@ -74,7 +74,8 @@ public class ExistsQueryBuilder extends AbstractQueryBuilder<ExistsQueryBuilder> } public static Query newFilter(QueryShardContext context, String fieldPattern) { - final FieldNamesFieldMapper.FieldNamesFieldType fieldNamesFieldType = (FieldNamesFieldMapper.FieldNamesFieldType)context.getMapperService().fullName(FieldNamesFieldMapper.NAME); + final FieldNamesFieldMapper.FieldNamesFieldType fieldNamesFieldType = + (FieldNamesFieldMapper.FieldNamesFieldType)context.getMapperService().fullName(FieldNamesFieldMapper.NAME); if (fieldNamesFieldType == null) { // can only happen when no types exist, so no docs exist either return Queries.newMatchNoDocsQuery(); diff --git a/core/src/main/java/org/elasticsearch/index/query/ExistsQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/ExistsQueryParser.java index 86a5311ff4..2c2d5e3786 100644 --- a/core/src/main/java/org/elasticsearch/index/query/ExistsQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/ExistsQueryParser.java @@ -30,14 +30,10 @@ import java.io.IOException; */ public class ExistsQueryParser implements QueryParser<ExistsQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(ExistsQueryBuilder.NAME); public static final ParseField FIELD_FIELD = new ParseField("field"); @Override - public String[] names() { - return new String[]{ExistsQueryBuilder.NAME}; - } - - @Override public ExistsQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -58,10 +54,12 @@ public class ExistsQueryParser implements QueryParser<ExistsQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + ExistsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + ExistsQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + ExistsQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + ExistsQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryBuilder.java index 9ce592ca4c..0ecc3278aa 100644 --- a/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryBuilder.java @@ -31,7 +31,8 @@ import org.elasticsearch.index.mapper.MappedFieldType; import java.io.IOException; import java.util.Objects; -public class FieldMaskingSpanQueryBuilder extends AbstractQueryBuilder<FieldMaskingSpanQueryBuilder> implements SpanQueryBuilder<FieldMaskingSpanQueryBuilder>{ +public class FieldMaskingSpanQueryBuilder extends AbstractQueryBuilder<FieldMaskingSpanQueryBuilder> + implements SpanQueryBuilder<FieldMaskingSpanQueryBuilder>{ public static final String NAME = "field_masking_span"; @@ -39,7 +40,8 @@ public class FieldMaskingSpanQueryBuilder extends AbstractQueryBuilder<FieldMask private final String fieldName; - static final FieldMaskingSpanQueryBuilder PROTOTYPE = new FieldMaskingSpanQueryBuilder(new SpanTermQueryBuilder("field", "text"), "field"); + static final FieldMaskingSpanQueryBuilder PROTOTYPE = + new FieldMaskingSpanQueryBuilder(new SpanTermQueryBuilder("field", "text"), "field"); /** * Constructs a new {@link FieldMaskingSpanQueryBuilder} given an inner {@link SpanQueryBuilder} for diff --git a/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryParser.java index de8b7b97b2..7a0b7f1154 100644 --- a/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/FieldMaskingSpanQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,15 +30,11 @@ import java.io.IOException; */ public class FieldMaskingSpanQueryParser implements QueryParser<FieldMaskingSpanQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(FieldMaskingSpanQueryBuilder.NAME); public static final ParseField FIELD_FIELD = new ParseField("field"); public static final ParseField QUERY_FIELD = new ParseField("query"); @Override - public String[] names() { - return new String[]{FieldMaskingSpanQueryBuilder.NAME, Strings.toCamelCase(FieldMaskingSpanQueryBuilder.NAME)}; - } - - @Override public FieldMaskingSpanQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -73,7 +68,8 @@ public class FieldMaskingSpanQueryParser implements QueryParser<FieldMaskingSpan } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[field_masking_span] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[field_masking_span] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/FuzzyQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/FuzzyQueryParser.java index 55dddbe4af..e81c15d819 100644 --- a/core/src/main/java/org/elasticsearch/index/query/FuzzyQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/FuzzyQueryParser.java @@ -33,6 +33,7 @@ import java.io.IOException; @Deprecated public class FuzzyQueryParser implements QueryParser<FuzzyQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(FuzzyQueryBuilder.NAME); public static final ParseField TERM_FIELD = new ParseField("term"); public static final ParseField VALUE_FIELD = new ParseField("value"); public static final ParseField PREFIX_LENGTH_FIELD = new ParseField("prefix_length"); @@ -41,11 +42,6 @@ public class FuzzyQueryParser implements QueryParser<FuzzyQueryBuilder> { public static final ParseField REWRITE_FIELD = new ParseField("rewrite"); @Override - public String[] names() { - return new String[]{ FuzzyQueryBuilder.NAME }; - } - - @Override public FuzzyQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryBuilder.java index 2c906dc7cb..71238e1b85 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryBuilder.java @@ -48,7 +48,7 @@ import java.util.Objects; * */ public class GeoBoundingBoxQueryBuilder extends AbstractQueryBuilder<GeoBoundingBoxQueryBuilder> { /** Name of the query. */ - public static final String NAME = "geo_bbox"; + public static final String NAME = "geo_bounding_box"; /** Default type for executing this query (memory as of this writing). */ public static final GeoExecType DEFAULT_TYPE = GeoExecType.MEMORY; /** Needed for serialization. */ diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryParser.java index e2bb6e7dcb..2ac526b1f1 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoBoundingBoxQueryParser.java @@ -30,8 +30,7 @@ import java.io.IOException; public class GeoBoundingBoxQueryParser implements QueryParser<GeoBoundingBoxQueryBuilder> { - public static final String NAME = "geo_bbox"; - + public static final ParseField QUERY_NAME_FIELD = new ParseField(GeoBoundingBoxQueryBuilder.NAME, "geo_bbox"); public static final ParseField IGNORE_MALFORMED_FIELD = new ParseField("ignore_malformed"); public static final ParseField TYPE_FIELD = new ParseField("type"); public static final ParseField VALIDATION_METHOD_FIELD = new ParseField("validation_method"); @@ -47,11 +46,6 @@ public class GeoBoundingBoxQueryParser implements QueryParser<GeoBoundingBoxQuer public static final ParseField BOTTOM_LEFT_FIELD = new ParseField("bottom_left"); @Override - public String[] names() { - return new String[]{GeoBoundingBoxQueryBuilder.NAME, "geoBbox", "geo_bounding_box", "geoBoundingBox"}; - } - - @Override public GeoBoundingBoxQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -114,11 +108,13 @@ public class GeoBoundingBoxQueryParser implements QueryParser<GeoBoundingBoxQuer bottom = sparse.getLat(); left = sparse.getLon(); } else { - throw new ElasticsearchParseException("failed to parse [{}] query. unexpected field [{}]", NAME, currentFieldName); + throw new ElasticsearchParseException("failed to parse [{}] query. unexpected field [{}]", + QUERY_NAME_FIELD.getPreferredName(), currentFieldName); } } } else { - throw new ElasticsearchParseException("failed to parse [{}] query. field name expected but [{}] found", NAME, token); + throw new ElasticsearchParseException("failed to parse [{}] query. field name expected but [{}] found", + QUERY_NAME_FIELD.getPreferredName(), token); } } } else if (token.isValue()) { @@ -138,7 +134,8 @@ public class GeoBoundingBoxQueryParser implements QueryParser<GeoBoundingBoxQuer } else if (parseContext.parseFieldMatcher().match(currentFieldName, IGNORE_MALFORMED_FIELD)) { ignoreMalformed = parser.booleanValue(); } else { - throw new ParsingException(parser.getTokenLocation(), "failed to parse [{}] query. unexpected field [{}]", NAME, currentFieldName); + throw new ParsingException(parser.getTokenLocation(), "failed to parse [{}] query. unexpected field [{}]", + QUERY_NAME_FIELD.getPreferredName(), currentFieldName); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryBuilder.java index b11b57df17..67f07b92e5 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryBuilder.java @@ -234,7 +234,8 @@ public class GeoDistanceQueryBuilder extends AbstractQueryBuilder<GeoDistanceQue if (indexVersionCreated.before(Version.V_2_2_0)) { GeoPointFieldMapperLegacy.GeoPointFieldType geoFieldType = ((GeoPointFieldMapperLegacy.GeoPointFieldType) fieldType); IndexGeoPointFieldData indexFieldData = shardContext.getForField(fieldType); - return new GeoDistanceRangeQuery(center, null, normDistance, true, false, geoDistance, geoFieldType, indexFieldData, optimizeBbox); + return new GeoDistanceRangeQuery(center, null, normDistance, true, false, geoDistance, + geoFieldType, indexFieldData, optimizeBbox); } // if index created V_2_2 use (soon to be legacy) numeric encoding postings format diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryParser.java index 3828f78690..66b51961f3 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryParser.java @@ -42,6 +42,7 @@ import java.io.IOException; */ public class GeoDistanceQueryParser implements QueryParser<GeoDistanceQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(GeoDistanceQueryBuilder.NAME); public static final ParseField VALIDATION_METHOD_FIELD = new ParseField("validation_method"); public static final ParseField IGNORE_MALFORMED_FIELD = new ParseField("ignore_malformed"); public static final ParseField COERCE_FIELD = new ParseField("coerce", "normalize"); @@ -51,11 +52,6 @@ public class GeoDistanceQueryParser implements QueryParser<GeoDistanceQueryBuild public static final ParseField DISTANCE_FIELD = new ParseField("distance"); @Override - public String[] names() { - return new String[]{GeoDistanceQueryBuilder.NAME, "geoDistance"}; - } - - @Override public GeoDistanceQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -98,8 +94,8 @@ public class GeoDistanceQueryParser implements QueryParser<GeoDistanceQueryBuild } else if (currentName.equals(GeoPointFieldMapper.Names.GEOHASH)) { point.resetFromGeoHash(parser.text()); } else { - throw new ParsingException(parser.getTokenLocation(), "[geo_distance] query does not support [" + currentFieldName - + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[geo_distance] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceRangeQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceRangeQueryParser.java index 77799fa627..f85f893773 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceRangeQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceRangeQueryParser.java @@ -40,6 +40,7 @@ import java.io.IOException; */ public class GeoDistanceRangeQueryParser implements QueryParser<GeoDistanceRangeQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(GeoDistanceRangeQueryBuilder.NAME); public static final ParseField FROM_FIELD = new ParseField("from"); public static final ParseField TO_FIELD = new ParseField("to"); public static final ParseField INCLUDE_LOWER_FIELD = new ParseField("include_lower"); @@ -58,11 +59,6 @@ public class GeoDistanceRangeQueryParser implements QueryParser<GeoDistanceRange public static final ParseField VALIDATION_METHOD = new ParseField("validation_method"); @Override - public String[] names() { - return new String[]{GeoDistanceRangeQueryBuilder.NAME, "geoDistanceRange"}; - } - - @Override public GeoDistanceRangeQueryBuilder getBuilderPrototype() { return GeoDistanceRangeQueryBuilder.PROTOTYPE; } @@ -173,7 +169,8 @@ public class GeoDistanceRangeQueryParser implements QueryParser<GeoDistanceRange } else if (parseContext.parseFieldMatcher().match(currentFieldName, DISTANCE_TYPE_FIELD)) { geoDistance = GeoDistance.fromString(parser.text()); } else if (currentFieldName.endsWith(GeoPointFieldMapper.Names.LAT_SUFFIX)) { - String maybeFieldName = currentFieldName.substring(0, currentFieldName.length() - GeoPointFieldMapper.Names.LAT_SUFFIX.length()); + String maybeFieldName = currentFieldName.substring(0, + currentFieldName.length() - GeoPointFieldMapper.Names.LAT_SUFFIX.length()); if (fieldName == null || fieldName.equals(maybeFieldName)) { fieldName = maybeFieldName; } else { @@ -185,7 +182,8 @@ public class GeoDistanceRangeQueryParser implements QueryParser<GeoDistanceRange } point.resetLat(parser.doubleValue()); } else if (currentFieldName.endsWith(GeoPointFieldMapper.Names.LON_SUFFIX)) { - String maybeFieldName = currentFieldName.substring(0, currentFieldName.length() - GeoPointFieldMapper.Names.LON_SUFFIX.length()); + String maybeFieldName = currentFieldName.substring(0, + currentFieldName.length() - GeoPointFieldMapper.Names.LON_SUFFIX.length()); if (fieldName == null || fieldName.equals(maybeFieldName)) { fieldName = maybeFieldName; } else { diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryBuilder.java index b251728a8a..fd0528a7a3 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryBuilder.java @@ -44,8 +44,8 @@ public class GeoPolygonQueryBuilder extends AbstractQueryBuilder<GeoPolygonQuery public static final String NAME = "geo_polygon"; - private static final List<GeoPoint> PROTO_SHAPE = Arrays.asList(new GeoPoint[] { new GeoPoint(1.0, 1.0), new GeoPoint(1.0, 2.0), - new GeoPoint(2.0, 1.0) }); + private static final List<GeoPoint> PROTO_SHAPE = Arrays.asList(new GeoPoint(1.0, 1.0), new GeoPoint(1.0, 2.0), + new GeoPoint(2.0, 1.0)); static final GeoPolygonQueryBuilder PROTOTYPE = new GeoPolygonQueryBuilder("field", PROTO_SHAPE); @@ -171,7 +171,8 @@ public class GeoPolygonQueryBuilder extends AbstractQueryBuilder<GeoPolygonQuery builder.endObject(); builder.field(GeoPolygonQueryParser.COERCE_FIELD.getPreferredName(), GeoValidationMethod.isCoerce(validationMethod)); - builder.field(GeoPolygonQueryParser.IGNORE_MALFORMED_FIELD.getPreferredName(), GeoValidationMethod.isIgnoreMalformed(validationMethod)); + builder.field(GeoPolygonQueryParser.IGNORE_MALFORMED_FIELD.getPreferredName(), + GeoValidationMethod.isIgnoreMalformed(validationMethod)); printBoostAndQueryName(builder); builder.endObject(); diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryParser.java index 3298a0fdae..325fb504e3 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoPolygonQueryParser.java @@ -44,17 +44,13 @@ import java.util.List; */ public class GeoPolygonQueryParser implements QueryParser<GeoPolygonQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(GeoPolygonQueryBuilder.NAME); public static final ParseField COERCE_FIELD = new ParseField("coerce", "normalize"); public static final ParseField IGNORE_MALFORMED_FIELD = new ParseField("ignore_malformed"); public static final ParseField VALIDATION_METHOD = new ParseField("validation_method"); public static final ParseField POINTS_FIELD = new ParseField("points"); @Override - public String[] names() { - return new String[]{GeoPolygonQueryBuilder.NAME, "geoPolygon"}; - } - - @Override public GeoPolygonQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -88,12 +84,12 @@ public class GeoPolygonQueryParser implements QueryParser<GeoPolygonQueryBuilder shell.add(GeoUtils.parseGeoPoint(parser)); } } else { - throw new ParsingException(parser.getTokenLocation(), "[geo_polygon] query does not support [" + currentFieldName - + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[geo_polygon] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[geo_polygon] query does not support token type [" + token.name() - + "] under [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[geo_polygon] query does not support token type [" + token.name() + "] under [" + currentFieldName + "]"); } } } else if (token.isValue()) { @@ -103,7 +99,7 @@ public class GeoPolygonQueryParser implements QueryParser<GeoPolygonQueryBuilder boost = parser.floatValue(); } else if (parseContext.parseFieldMatcher().match(currentFieldName, COERCE_FIELD)) { coerce = parser.booleanValue(); - if (coerce == true) { + if (coerce) { ignoreMalformed = true; } } else if (parseContext.parseFieldMatcher().match(currentFieldName, IGNORE_MALFORMED_FIELD)) { @@ -111,7 +107,8 @@ public class GeoPolygonQueryParser implements QueryParser<GeoPolygonQueryBuilder } else if (parseContext.parseFieldMatcher().match(currentFieldName, VALIDATION_METHOD)) { validationMethod = GeoValidationMethod.fromString(parser.text()); } else { - throw new ParsingException(parser.getTokenLocation(), "[geo_polygon] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[geo_polygon] query does not support [" + currentFieldName + "]"); } } else { throw new ParsingException(parser.getTokenLocation(), "[geo_polygon] unexpected token type [" + token.name() + "]"); diff --git a/core/src/main/java/org/elasticsearch/index/query/GeoShapeQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/GeoShapeQueryParser.java index c1d040f33b..2a97df96b4 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoShapeQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoShapeQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.geo.ShapeRelation; import org.elasticsearch.common.geo.SpatialStrategy; import org.elasticsearch.common.geo.builders.ShapeBuilder; @@ -31,6 +30,7 @@ import java.io.IOException; public class GeoShapeQueryParser implements QueryParser<GeoShapeQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(GeoShapeQueryBuilder.NAME); public static final ParseField SHAPE_FIELD = new ParseField("shape"); public static final ParseField STRATEGY_FIELD = new ParseField("strategy"); public static final ParseField RELATION_FIELD = new ParseField("relation"); @@ -41,11 +41,6 @@ public class GeoShapeQueryParser implements QueryParser<GeoShapeQueryBuilder> { public static final ParseField SHAPE_PATH_FIELD = new ParseField("path"); @Override - public String[] names() { - return new String[]{GeoShapeQueryBuilder.NAME, Strings.toCamelCase(GeoShapeQueryBuilder.NAME)}; - } - - @Override public GeoShapeQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -69,7 +64,8 @@ public class GeoShapeQueryParser implements QueryParser<GeoShapeQueryBuilder> { currentFieldName = parser.currentName(); } else if (token == XContentParser.Token.START_OBJECT) { if (fieldName != null) { - throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + "] point specified twice. [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + + GeoShapeQueryBuilder.NAME + "] point specified twice. [" + currentFieldName + "]"); } fieldName = currentFieldName; while ((token = parser.nextToken()) != XContentParser.Token.END_OBJECT) { @@ -104,11 +100,13 @@ public class GeoShapeQueryParser implements QueryParser<GeoShapeQueryBuilder> { shapePath = parser.text(); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } } @@ -118,7 +116,8 @@ public class GeoShapeQueryParser implements QueryParser<GeoShapeQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + GeoShapeQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/GeohashCellQuery.java b/core/src/main/java/org/elasticsearch/index/query/GeohashCellQuery.java index 9f0d259374..1f5aa537c0 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeohashCellQuery.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeohashCellQuery.java @@ -75,7 +75,8 @@ public class GeohashCellQuery { * @param geohashes optional array of additional geohashes * @return a new GeoBoundinboxfilter */ - public static Query create(QueryShardContext context, BaseGeoPointFieldMapper.GeoPointFieldType fieldType, String geohash, @Nullable List<CharSequence> geohashes) { + public static Query create(QueryShardContext context, BaseGeoPointFieldMapper.GeoPointFieldType fieldType, + String geohash, @Nullable List<CharSequence> geohashes) { MappedFieldType geoHashMapper = fieldType.geoHashFieldType(); if (geoHashMapper == null) { throw new IllegalArgumentException("geohash filter needs geohash_prefix to be enabled"); @@ -270,16 +271,13 @@ public class GeohashCellQuery { public static class Parser implements QueryParser<Builder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(NAME); + @Inject public Parser() { } @Override - public String[] names() { - return new String[]{NAME, Strings.toCamelCase(NAME)}; - } - - @Override public Builder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/HasChildQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/HasChildQueryBuilder.java index a58f954ea1..48fde6a935 100644 --- a/core/src/main/java/org/elasticsearch/index/query/HasChildQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/HasChildQueryBuilder.java @@ -77,7 +77,8 @@ public class HasChildQueryBuilder extends AbstractQueryBuilder<HasChildQueryBuil static final HasChildQueryBuilder PROTOTYPE = new HasChildQueryBuilder("", EmptyQueryBuilder.PROTOTYPE); - public HasChildQueryBuilder(String type, QueryBuilder query, int maxChildren, int minChildren, ScoreMode scoreMode, InnerHitBuilder innerHitBuilder) { + public HasChildQueryBuilder(String type, QueryBuilder query, int maxChildren, int minChildren, ScoreMode scoreMode, + InnerHitBuilder innerHitBuilder) { this(type, query); scoreMode(scoreMode); this.maxChildren = maxChildren; @@ -288,7 +289,8 @@ public class HasChildQueryBuilder extends AbstractQueryBuilder<HasChildQueryBuil indexSearcher.setSimilarity(similarity); IndexParentChildFieldData indexParentChildFieldData = parentChildIndexFieldData.loadGlobal((DirectoryReader) reader); MultiDocValues.OrdinalMap ordinalMap = ParentChildIndexFieldData.getOrdinalMap(indexParentChildFieldData, parentType); - return JoinUtil.createJoinQuery(joinField, innerQuery, toQuery, indexSearcher, scoreMode, ordinalMap, minChildren, maxChildren); + return JoinUtil.createJoinQuery(joinField, innerQuery, toQuery, indexSearcher, scoreMode, + ordinalMap, minChildren, maxChildren); } else { if (reader.leaves().isEmpty() && reader.numDocs() == 0) { // asserting reader passes down a MultiReader during rewrite which makes this @@ -297,7 +299,8 @@ public class HasChildQueryBuilder extends AbstractQueryBuilder<HasChildQueryBuil // and rewrite to match nothing return new MatchNoDocsQuery(); } - throw new IllegalStateException("can't load global ordinals for reader of type: " + reader.getClass() + " must be a DirectoryReader"); + throw new IllegalStateException("can't load global ordinals for reader of type: " + + reader.getClass() + " must be a DirectoryReader"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/HasChildQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/HasChildQueryParser.java index 667731e0fc..61d0da308f 100644 --- a/core/src/main/java/org/elasticsearch/index/query/HasChildQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/HasChildQueryParser.java @@ -22,7 +22,6 @@ package org.elasticsearch.index.query; import org.apache.lucene.search.join.ScoreMode; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import org.elasticsearch.index.query.support.InnerHitBuilder; @@ -34,6 +33,7 @@ import java.util.Locale; */ public class HasChildQueryParser implements QueryParser<HasChildQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(HasChildQueryBuilder.NAME); public static final ParseField QUERY_FIELD = new ParseField("query", "filter"); public static final ParseField TYPE_FIELD = new ParseField("type", "child_type"); public static final ParseField MAX_CHILDREN_FIELD = new ParseField("max_children"); @@ -42,11 +42,6 @@ public class HasChildQueryParser implements QueryParser<HasChildQueryBuilder> { public static final ParseField INNER_HITS_FIELD = new ParseField("inner_hits"); @Override - public String[] names() { - return new String[] { HasChildQueryBuilder.NAME, Strings.toCamelCase(HasChildQueryBuilder.NAME) }; - } - - @Override public HasChildQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); float boost = AbstractQueryBuilder.DEFAULT_BOOST; @@ -90,7 +85,8 @@ public class HasChildQueryParser implements QueryParser<HasChildQueryBuilder> { } } } - HasChildQueryBuilder hasChildQueryBuilder = new HasChildQueryBuilder(childType, iqb, maxChildren, minChildren, scoreMode, innerHitBuilder); + HasChildQueryBuilder hasChildQueryBuilder = new HasChildQueryBuilder(childType, iqb, maxChildren, minChildren, + scoreMode, innerHitBuilder); hasChildQueryBuilder.queryName(queryName); hasChildQueryBuilder.boost(boost); return hasChildQueryBuilder; diff --git a/core/src/main/java/org/elasticsearch/index/query/HasParentQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/HasParentQueryParser.java index 4554184c94..1b4f28dd53 100644 --- a/core/src/main/java/org/elasticsearch/index/query/HasParentQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/HasParentQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import org.elasticsearch.index.query.support.InnerHitBuilder; @@ -30,19 +29,14 @@ import java.io.IOException; public class HasParentQueryParser implements QueryParser<HasParentQueryBuilder> { private static final HasParentQueryBuilder PROTOTYPE = new HasParentQueryBuilder("", EmptyQueryBuilder.PROTOTYPE); + public static final ParseField QUERY_NAME_FIELD = new ParseField(HasParentQueryBuilder.NAME); public static final ParseField QUERY_FIELD = new ParseField("query", "filter"); - //public static final ParseField SCORE_MODE_FIELD = new ParseField("score_mode").withAllDeprecated("score"); public static final ParseField SCORE_MODE_FIELD = new ParseField("score_mode").withAllDeprecated("score"); public static final ParseField TYPE_FIELD = new ParseField("parent_type", "type"); public static final ParseField SCORE_FIELD = new ParseField("score"); public static final ParseField INNER_HITS_FIELD = new ParseField("inner_hits"); @Override - public String[] names() { - return new String[]{HasParentQueryBuilder.NAME, Strings.toCamelCase(HasParentQueryBuilder.NAME)}; - } - - @Override public HasParentQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); float boost = AbstractQueryBuilder.DEFAULT_BOOST; @@ -75,7 +69,8 @@ public class HasParentQueryParser implements QueryParser<HasParentQueryBuilder> } else if ("none".equals(scoreModeValue)) { score = false; } else { - throw new ParsingException(parser.getTokenLocation(), "[has_parent] query does not support [" + scoreModeValue + "] as an option for score_mode"); + throw new ParsingException(parser.getTokenLocation(), "[has_parent] query does not support [" + + scoreModeValue + "] as an option for score_mode"); } } else if (parseContext.parseFieldMatcher().match(currentFieldName, SCORE_FIELD)) { score = parser.booleanValue(); diff --git a/core/src/main/java/org/elasticsearch/index/query/IdsQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/IdsQueryParser.java index 46058d98eb..bc60eb77fb 100644 --- a/core/src/main/java/org/elasticsearch/index/query/IdsQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/IdsQueryParser.java @@ -33,15 +33,10 @@ import java.util.List; */ public class IdsQueryParser implements QueryParser<IdsQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(IdsQueryBuilder.NAME); public static final ParseField TYPE_FIELD = new ParseField("type", "types", "_type"); - public static final ParseField VALUES_FIELD = new ParseField("values"); - @Override - public String[] names() { - return new String[]{IdsQueryBuilder.NAME}; - } - /** * @return a QueryBuilder representation of the query passed in as XContent in the parse context */ @@ -71,8 +66,8 @@ public class IdsQueryParser implements QueryParser<IdsQueryBuilder> { } ids.add(id); } else { - throw new ParsingException(parser.getTokenLocation(), "Illegal value for id, expecting a string or number, got: " - + token); + throw new ParsingException(parser.getTokenLocation(), + "Illegal value for id, expecting a string or number, got: " + token); } } } else if (parseContext.parseFieldMatcher().match(currentFieldName, TYPE_FIELD)) { @@ -84,7 +79,8 @@ public class IdsQueryParser implements QueryParser<IdsQueryBuilder> { types.add(value); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + IdsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + IdsQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else if (token.isValue()) { if (parseContext.parseFieldMatcher().match(currentFieldName, TYPE_FIELD)) { @@ -94,10 +90,12 @@ public class IdsQueryParser implements QueryParser<IdsQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + IdsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + IdsQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + IdsQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + IdsQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } if (!idsProvided) { diff --git a/core/src/main/java/org/elasticsearch/index/query/IndicesQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/IndicesQueryParser.java index d66dcdbc22..cc261e5267 100644 --- a/core/src/main/java/org/elasticsearch/index/query/IndicesQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/IndicesQueryParser.java @@ -32,16 +32,11 @@ import java.util.Collection; */ public class IndicesQueryParser implements QueryParser { + public static final ParseField QUERY_NAME_FIELD = new ParseField(IndicesQueryBuilder.NAME); public static final ParseField QUERY_FIELD = new ParseField("query"); public static final ParseField NO_MATCH_QUERY = new ParseField("no_match_query"); public static final ParseField INDEX_FIELD = new ParseField("index"); public static final ParseField INDICES_FIELD = new ParseField("indices"); - - - @Override - public String[] names() { - return new String[]{IndicesQueryBuilder.NAME}; - } @Override public QueryBuilder fromXContent(QueryParseContext parseContext) throws IOException, ParsingException { diff --git a/core/src/main/java/org/elasticsearch/index/query/MatchAllQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/MatchAllQueryParser.java index b72e80be86..882c2a1b26 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MatchAllQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/MatchAllQueryParser.java @@ -19,8 +19,8 @@ package org.elasticsearch.index.query; +import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -30,10 +30,7 @@ import java.io.IOException; */ public class MatchAllQueryParser implements QueryParser<MatchAllQueryBuilder> { - @Override - public String[] names() { - return new String[]{MatchAllQueryBuilder.NAME, Strings.toCamelCase(MatchAllQueryBuilder.NAME)}; - } + public static final ParseField QUERY_NAME_FIELD = new ParseField(MatchAllQueryBuilder.NAME); @Override public MatchAllQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { @@ -52,10 +49,12 @@ public class MatchAllQueryParser implements QueryParser<MatchAllQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + MatchAllQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + MatchAllQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + MatchAllQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + MatchAllQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } MatchAllQueryBuilder queryBuilder = new MatchAllQueryBuilder(); diff --git a/core/src/main/java/org/elasticsearch/index/query/MatchNoneQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/MatchNoneQueryParser.java index 70fc74d66e..466c8ac506 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MatchNoneQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/MatchNoneQueryParser.java @@ -19,18 +19,15 @@ package org.elasticsearch.index.query; +import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; public class MatchNoneQueryParser implements QueryParser<MatchNoneQueryBuilder> { - @Override - public String[] names() { - return new String[]{MatchNoneQueryBuilder.NAME, Strings.toCamelCase(MatchNoneQueryBuilder.NAME)}; - } + public static final ParseField QUERY_NAME_FIELD = new ParseField(MatchNoneQueryBuilder.NAME); @Override public MatchNoneQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { @@ -49,10 +46,12 @@ public class MatchNoneQueryParser implements QueryParser<MatchNoneQueryBuilder> } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); } else { - throw new ParsingException(parser.getTokenLocation(), "["+MatchNoneQueryBuilder.NAME+"] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "["+MatchNoneQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + MatchNoneQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + MatchNoneQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/MatchPhrasePrefixQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/MatchPhrasePrefixQueryBuilder.java index c342fe5f39..d07f782de9 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MatchPhrasePrefixQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/MatchPhrasePrefixQueryBuilder.java @@ -38,9 +38,9 @@ import java.util.Objects; */ public class MatchPhrasePrefixQueryBuilder extends AbstractQueryBuilder<MatchPhrasePrefixQueryBuilder> { - public static final ParseField MAX_EXPANSIONS_FIELD = new ParseField("max_expansions"); - public static final String NAME = "match_phrase_prefix"; + public static final ParseField QUERY_NAME_FIELD = new ParseField(NAME); + public static final ParseField MAX_EXPANSIONS_FIELD = new ParseField("max_expansions"); public static final MatchPhrasePrefixQueryBuilder PROTOTYPE = new MatchPhrasePrefixQueryBuilder("", ""); diff --git a/core/src/main/java/org/elasticsearch/index/query/MatchPhraseQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/MatchPhraseQueryBuilder.java index bee91f5f21..e1e28a9dc2 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MatchPhraseQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/MatchPhraseQueryBuilder.java @@ -36,9 +36,9 @@ import java.util.Objects; */ public class MatchPhraseQueryBuilder extends AbstractQueryBuilder<MatchPhraseQueryBuilder> { - public static final ParseField SLOP_FIELD = new ParseField("slop", "phrase_slop"); - public static final String NAME = "match_phrase"; + public static final ParseField QUERY_NAME_FIELD = new ParseField(NAME); + public static final ParseField SLOP_FIELD = new ParseField("slop", "phrase_slop"); public static final MatchPhraseQueryBuilder PROTOTYPE = new MatchPhraseQueryBuilder("", ""); diff --git a/core/src/main/java/org/elasticsearch/index/query/MatchQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/MatchQueryBuilder.java index d60cebede8..793c2af7a7 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MatchQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/MatchQueryBuilder.java @@ -61,6 +61,8 @@ public class MatchQueryBuilder extends AbstractQueryBuilder<MatchQueryBuilder> { /** The name for the match query */ public static final String NAME = "match"; + public static final ParseField QUERY_NAME_FIELD = new ParseField(NAME, "match_fuzzy", "fuzzy_match"); + /** The default mode terms are combined in a match query */ public static final Operator DEFAULT_OPERATOR = Operator.OR; diff --git a/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryBuilder.java index 242af5af05..8b09ba7204 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryBuilder.java @@ -79,7 +79,7 @@ import static org.elasticsearch.index.mapper.Uid.createUidAsBytes; */ public class MoreLikeThisQueryBuilder extends AbstractQueryBuilder<MoreLikeThisQueryBuilder> { - public static final String NAME = "mlt"; + public static final String NAME = "more_like_this"; public static final int DEFAULT_MAX_QUERY_TERMS = XMoreLikeThis.DEFAULT_MAX_QUERY_TERMS; public static final int DEFAULT_MIN_TERM_FREQ = XMoreLikeThis.DEFAULT_MIN_TERM_FREQ; @@ -478,10 +478,10 @@ public class MoreLikeThisQueryBuilder extends AbstractQueryBuilder<MoreLikeThisQ public MoreLikeThisQueryBuilder(@Nullable String[] fields, @Nullable String[] likeTexts, @Nullable Item[] likeItems) { // TODO we allow null here for the _all field, but this is forbidden in the parser. Re-check if (fields != null && fields.length == 0) { - throw new IllegalArgumentException("mlt query requires 'fields' to be specified"); + throw new IllegalArgumentException(NAME + " query requires 'fields' to be specified"); } if ((likeTexts == null || likeTexts.length == 0) && (likeItems == null || likeItems.length == 0)) { - throw new IllegalArgumentException("mlt query requires either 'like' texts or items to be specified."); + throw new IllegalArgumentException(NAME + " query requires either 'like' texts or items to be specified."); } this.fields = fields; this.likeTexts = Optional.ofNullable(likeTexts).orElse(Strings.EMPTY_ARRAY); @@ -811,7 +811,7 @@ public class MoreLikeThisQueryBuilder extends AbstractQueryBuilder<MoreLikeThisQ if (moreLikeFields.isEmpty()) { return null; } - mltQuery.setMoreLikeFields(moreLikeFields.toArray(Strings.EMPTY_ARRAY)); + mltQuery.setMoreLikeFields(moreLikeFields.toArray(new String[moreLikeFields.size()])); // handle like texts if (likeTexts.length > 0) { @@ -950,7 +950,7 @@ public class MoreLikeThisQueryBuilder extends AbstractQueryBuilder<MoreLikeThisQ uids.add(createUidAsBytes(item.type(), item.id())); } if (!uids.isEmpty()) { - TermsQuery query = new TermsQuery(UidFieldMapper.NAME, uids.toArray(new BytesRef[0])); + TermsQuery query = new TermsQuery(UidFieldMapper.NAME, uids.toArray(new BytesRef[uids.size()])); boolQuery.add(query, BooleanClause.Occur.MUST_NOT); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryParser.java index bc629d8382..c02f1bef9b 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryParser.java @@ -35,6 +35,8 @@ import java.util.List; */ public class MoreLikeThisQueryParser implements QueryParser<MoreLikeThisQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(MoreLikeThisQueryBuilder.NAME, "mlt"); + public interface Field { ParseField FIELDS = new ParseField("fields"); ParseField LIKE = new ParseField("like"); @@ -57,11 +59,6 @@ public class MoreLikeThisQueryParser implements QueryParser<MoreLikeThisQueryBui } @Override - public String[] names() { - return new String[]{MoreLikeThisQueryBuilder.NAME, "more_like_this", "moreLikeThis"}; - } - - @Override public MoreLikeThisQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryParser.java index bb3ef5fa17..d367986c47 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryParser.java @@ -34,6 +34,7 @@ import java.util.Map; */ public class MultiMatchQueryParser implements QueryParser<MultiMatchQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(MultiMatchQueryBuilder.NAME); public static final ParseField SLOP_FIELD = new ParseField("slop", "phrase_slop"); public static final ParseField ZERO_TERMS_QUERY_FIELD = new ParseField("zero_terms_query"); public static final ParseField LENIENT_FIELD = new ParseField("lenient"); @@ -51,13 +52,6 @@ public class MultiMatchQueryParser implements QueryParser<MultiMatchQueryBuilder public static final ParseField FIELDS_FIELD = new ParseField("fields"); @Override - public String[] names() { - return new String[]{ - MultiMatchQueryBuilder.NAME, "multiMatch" - }; - } - - @Override public MultiMatchQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -94,7 +88,8 @@ public class MultiMatchQueryParser implements QueryParser<MultiMatchQueryBuilder } else if (token.isValue()) { parseFieldAndBoost(parser, fieldsBoosts); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + MultiMatchQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + MultiMatchQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else if (token.isValue()) { if (parseContext.parseFieldMatcher().match(currentFieldName, QUERY_FIELD)) { @@ -139,10 +134,12 @@ public class MultiMatchQueryParser implements QueryParser<MultiMatchQueryBuilder } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + MultiMatchQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + MultiMatchQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + MultiMatchQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + MultiMatchQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/NestedQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/NestedQueryParser.java index 0fb7649cf7..7dec49db8c 100644 --- a/core/src/main/java/org/elasticsearch/index/query/NestedQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/NestedQueryParser.java @@ -19,28 +19,24 @@ package org.elasticsearch.index.query; -import java.io.IOException; - import org.apache.lucene.search.join.ScoreMode; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; import org.elasticsearch.common.xcontent.XContentParser; import org.elasticsearch.index.query.support.InnerHitBuilder; +import java.io.IOException; + public class NestedQueryParser implements QueryParser<NestedQueryBuilder> { private static final NestedQueryBuilder PROTOTYPE = new NestedQueryBuilder("", EmptyQueryBuilder.PROTOTYPE); + public static final ParseField QUERY_NAME_FIELD = new ParseField(NestedQueryBuilder.NAME); public static final ParseField SCORE_MODE_FIELD = new ParseField("score_mode"); public static final ParseField PATH_FIELD = new ParseField("path"); public static final ParseField QUERY_FIELD = new ParseField("query"); public static final ParseField INNER_HITS_FIELD = new ParseField("inner_hits"); @Override - public String[] names() { - return new String[]{NestedQueryBuilder.NAME}; - } - - @Override public NestedQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); float boost = AbstractQueryBuilder.DEFAULT_BOOST; diff --git a/core/src/main/java/org/elasticsearch/index/query/Operator.java b/core/src/main/java/org/elasticsearch/index/query/Operator.java index 5b485c5673..528b792f19 100644 --- a/core/src/main/java/org/elasticsearch/index/query/Operator.java +++ b/core/src/main/java/org/elasticsearch/index/query/Operator.java @@ -71,6 +71,7 @@ public enum Operator implements Writeable<Operator> { } private static IllegalArgumentException newOperatorException(String op) { - return new IllegalArgumentException("operator needs to be either " + CollectionUtils.arrayAsArrayList(values()) + ", but not [" + op + "]"); + return new IllegalArgumentException("operator needs to be either " + + CollectionUtils.arrayAsArrayList(values()) + ", but not [" + op + "]"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/ParentIdQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/ParentIdQueryParser.java index 43788624c5..484551be34 100644 --- a/core/src/main/java/org/elasticsearch/index/query/ParentIdQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/ParentIdQueryParser.java @@ -27,15 +27,11 @@ import java.io.IOException; public final class ParentIdQueryParser implements QueryParser<ParentIdQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(ParentIdQueryBuilder.NAME); public static final ParseField ID_FIELD = new ParseField("id"); public static final ParseField TYPE_FIELD = new ParseField("type", "child_type"); @Override - public String[] names() { - return new String[]{ParentIdQueryBuilder.NAME}; - } - - @Override public ParentIdQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); float boost = AbstractQueryBuilder.DEFAULT_BOOST; diff --git a/core/src/main/java/org/elasticsearch/index/query/PercolatorQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/PercolatorQueryParser.java index a559db5992..2ae54359cb 100644 --- a/core/src/main/java/org/elasticsearch/index/query/PercolatorQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/PercolatorQueryParser.java @@ -30,6 +30,7 @@ import java.io.IOException; public class PercolatorQueryParser implements QueryParser<PercolatorQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(PercolatorQueryBuilder.NAME); public static final ParseField DOCUMENT_FIELD = new ParseField("document"); public static final ParseField DOCUMENT_TYPE_FIELD = new ParseField("document_type"); public static final ParseField INDEXED_DOCUMENT_FIELD_INDEX = new ParseField("index"); @@ -40,11 +41,6 @@ public class PercolatorQueryParser implements QueryParser<PercolatorQueryBuilder public static final ParseField INDEXED_DOCUMENT_FIELD_VERSION = new ParseField("version"); @Override - public String[] names() { - return new String[]{PercolatorQueryBuilder.NAME}; - } - - @Override public PercolatorQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); float boost = AbstractQueryBuilder.DEFAULT_BOOST; diff --git a/core/src/main/java/org/elasticsearch/index/query/PrefixQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/PrefixQueryParser.java index e13d937d84..5c3e203ea8 100644 --- a/core/src/main/java/org/elasticsearch/index/query/PrefixQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/PrefixQueryParser.java @@ -30,15 +30,11 @@ import java.io.IOException; */ public class PrefixQueryParser implements QueryParser<PrefixQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(PrefixQueryBuilder.NAME); public static final ParseField PREFIX_FIELD = new ParseField("value", "prefix"); public static final ParseField REWRITE_FIELD = new ParseField("rewrite"); @Override - public String[] names() { - return new String[]{PrefixQueryBuilder.NAME}; - } - - @Override public PrefixQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -70,7 +66,8 @@ public class PrefixQueryParser implements QueryParser<PrefixQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, REWRITE_FIELD)) { rewrite = parser.textOrNull(); } else { - throw new ParsingException(parser.getTokenLocation(), "[regexp] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[regexp] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/QueryParseContext.java b/core/src/main/java/org/elasticsearch/index/query/QueryParseContext.java index 439e6533de..e01310fa12 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryParseContext.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryParseContext.java @@ -117,12 +117,7 @@ public class QueryParseContext { if (token != XContentParser.Token.START_OBJECT && token != XContentParser.Token.START_ARRAY) { throw new ParsingException(parser.getTokenLocation(), "[_na] query malformed, no field after start_object"); } - - QueryParser queryParser = queryParser(queryName); - if (queryParser == null) { - throw new ParsingException(parser.getTokenLocation(), "No query registered for [" + queryName + "]"); - } - + QueryParser queryParser = indicesQueriesRegistry.getQueryParser(queryName, parseFieldMatcher, parser.getTokenLocation()); QueryBuilder result = queryParser.fromXContent(this); if (parser.currentToken() == XContentParser.Token.END_OBJECT || parser.currentToken() == XContentParser.Token.END_ARRAY) { // if we are at END_OBJECT, move to the next one... @@ -138,13 +133,4 @@ public class QueryParseContext { public void parser(XContentParser innerParser) { this.parser = innerParser; } - - /** - * Get the query parser for a specific type of query registered under its name - * @param name the name of the parser to retrieve - * @return the query parser - */ - private QueryParser queryParser(String name) { - return indicesQueriesRegistry.queryParsers().get(name); - } } diff --git a/core/src/main/java/org/elasticsearch/index/query/QueryParser.java b/core/src/main/java/org/elasticsearch/index/query/QueryParser.java index 74365d659a..1d394f9752 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryParser.java @@ -29,13 +29,6 @@ import java.io.IOException; public interface QueryParser<QB extends QueryBuilder<QB>> { /** - * The names this query parser is registered under. - */ - default String[] names() { // TODO remove this when nothing implements it - throw new UnsupportedOperationException(); - } - - /** * Creates a new {@link QueryBuilder} from the query held by the {@link QueryParseContext} * in {@link org.elasticsearch.common.xcontent.XContent} format * diff --git a/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryBuilder.java index fcab39b96e..ed5fde6a10 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryBuilder.java @@ -155,7 +155,8 @@ public class QueryStringQueryBuilder extends AbstractQueryBuilder<QueryStringQue } /** - * Adds a field to run the query string against. The field will be associated with the default boost of {@link AbstractQueryBuilder#DEFAULT_BOOST}. + * Adds a field to run the query string against. The field will be associated with the + * default boost of {@link AbstractQueryBuilder#DEFAULT_BOOST}. * Use {@link #field(String, float)} to set a specific boost for the field. */ public QueryStringQueryBuilder field(String field) { @@ -484,7 +485,8 @@ public class QueryStringQueryBuilder extends AbstractQueryBuilder<QueryStringQue builder.endArray(); builder.field(QueryStringQueryParser.USE_DIS_MAX_FIELD.getPreferredName(), this.useDisMax); builder.field(QueryStringQueryParser.TIE_BREAKER_FIELD.getPreferredName(), this.tieBreaker); - builder.field(QueryStringQueryParser.DEFAULT_OPERATOR_FIELD.getPreferredName(), this.defaultOperator.name().toLowerCase(Locale.ROOT)); + builder.field(QueryStringQueryParser.DEFAULT_OPERATOR_FIELD.getPreferredName(), + this.defaultOperator.name().toLowerCase(Locale.ROOT)); if (this.analyzer != null) { builder.field(QueryStringQueryParser.ANALYZER_FIELD.getPreferredName(), this.analyzer); } @@ -634,7 +636,8 @@ public class QueryStringQueryBuilder extends AbstractQueryBuilder<QueryStringQue Objects.equals(rewrite, other.rewrite) && Objects.equals(minimumShouldMatch, other.minimumShouldMatch) && Objects.equals(lenient, other.lenient) && - timeZone == null ? other.timeZone == null : other.timeZone != null && Objects.equals(timeZone.getID(), other.timeZone.getID()) && + timeZone == null ? other.timeZone == null : other.timeZone != null && + Objects.equals(timeZone.getID(), other.timeZone.getID()) && Objects.equals(escape, other.escape) && Objects.equals(maxDeterminizedStates, other.maxDeterminizedStates); } diff --git a/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryParser.java index f7d9d2989d..2c80062df3 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryStringQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.unit.Fuzziness; import org.elasticsearch.common.xcontent.XContentParser; @@ -35,6 +34,7 @@ import java.util.Map; */ public class QueryStringQueryParser implements QueryParser { + public static final ParseField QUERY_NAME_FIELD = new ParseField(QueryStringQueryBuilder.NAME); public static final ParseField QUERY_FIELD = new ParseField("query"); public static final ParseField FIELDS_FIELD = new ParseField("fields"); public static final ParseField DEFAULT_FIELD_FIELD = new ParseField("default_field"); @@ -62,11 +62,6 @@ public class QueryStringQueryParser implements QueryParser { public static final ParseField TIME_ZONE_FIELD = new ParseField("time_zone"); @Override - public String[] names() { - return new String[]{QueryStringQueryBuilder.NAME, Strings.toCamelCase(QueryStringQueryBuilder.NAME)}; - } - - @Override public QueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); String currentFieldName = null; @@ -123,7 +118,8 @@ public class QueryStringQueryParser implements QueryParser { fieldsAndWeights.put(fField, fBoost); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else if (token.isValue()) { if (parseContext.parseFieldMatcher().match(currentFieldName, QUERY_FIELD)) { @@ -181,15 +177,18 @@ public class QueryStringQueryParser implements QueryParser { try { timeZone = parser.text(); } catch (IllegalArgumentException e) { - throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + "] time_zone [" + parser.text() + "] is unknown"); + throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + + "] time_zone [" + parser.text() + "] is unknown"); } } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + QueryStringQueryBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } if (queryString == null) { diff --git a/core/src/main/java/org/elasticsearch/index/query/RangeQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/RangeQueryBuilder.java index 28f13dc8cf..fc3fbab3aa 100644 --- a/core/src/main/java/org/elasticsearch/index/query/RangeQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/RangeQueryBuilder.java @@ -270,7 +270,8 @@ public class RangeQueryBuilder extends AbstractQueryBuilder<RangeQueryBuilder> i return MappedFieldType.Relation.DISJOINT; } else { DateMathParser dateMathParser = format == null ? null : new DateMathParser(format); - return fieldType.isFieldWithinQuery(queryRewriteContext.getIndexReader(), from, to, includeLower, includeUpper, timeZone, dateMathParser); + return fieldType.isFieldWithinQuery(queryRewriteContext.getIndexReader(), from, to, includeLower, + includeUpper, timeZone, dateMathParser); } } @@ -308,7 +309,8 @@ public class RangeQueryBuilder extends AbstractQueryBuilder<RangeQueryBuilder> i if (this.format != null) { forcedDateParser = new DateMathParser(this.format); } - query = ((DateFieldMapper.DateFieldType) mapper).rangeQuery(from, to, includeLower, includeUpper, timeZone, forcedDateParser); + query = ((DateFieldMapper.DateFieldType) mapper).rangeQuery(from, to, includeLower, includeUpper, + timeZone, forcedDateParser); } else { if (timeZone != null) { throw new QueryShardException(context, "[range] time_zone can not be applied to non date field [" diff --git a/core/src/main/java/org/elasticsearch/index/query/RangeQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/RangeQueryParser.java index 10a13dd52f..bbf62f4796 100644 --- a/core/src/main/java/org/elasticsearch/index/query/RangeQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/RangeQueryParser.java @@ -30,8 +30,10 @@ import java.io.IOException; */ public class RangeQueryParser implements QueryParser<RangeQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(RangeQueryBuilder.NAME); public static final ParseField FIELDDATA_FIELD = new ParseField("fielddata").withAllDeprecated("[no replacement]"); - public static final ParseField NAME_FIELD = new ParseField("_name").withAllDeprecated("query name is not supported in short version of range query"); + public static final ParseField NAME_FIELD = new ParseField("_name") + .withAllDeprecated("query name is not supported in short version of range query"); public static final ParseField LTE_FIELD = new ParseField("lte", "le"); public static final ParseField GTE_FIELD = new ParseField("gte", "ge"); public static final ParseField FROM_FIELD = new ParseField("from"); @@ -44,11 +46,6 @@ public class RangeQueryParser implements QueryParser<RangeQueryBuilder> { public static final ParseField FORMAT_FIELD = new ParseField("format"); @Override - public String[] names() { - return new String[]{RangeQueryBuilder.NAME}; - } - - @Override public RangeQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -104,7 +101,8 @@ public class RangeQueryParser implements QueryParser<RangeQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[range] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[range] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/RegexpQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/RegexpQueryParser.java index 92305abc1d..ce0c3ba479 100644 --- a/core/src/main/java/org/elasticsearch/index/query/RegexpQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/RegexpQueryParser.java @@ -30,7 +30,9 @@ import java.io.IOException; */ public class RegexpQueryParser implements QueryParser<RegexpQueryBuilder> { - public static final ParseField NAME_FIELD = new ParseField("_name").withAllDeprecated("query name is not supported in short version of regexp query"); + public static final ParseField QUERY_NAME_FIELD = new ParseField(RegexpQueryBuilder.NAME); + public static final ParseField NAME_FIELD = new ParseField("_name") + .withAllDeprecated("query name is not supported in short version of regexp query"); public static final ParseField FLAGS_VALUE_FIELD = new ParseField("flags_value"); public static final ParseField MAX_DETERMINIZED_STATES_FIELD = new ParseField("max_determinized_states"); public static final ParseField FLAGS_FIELD = new ParseField("flags"); @@ -38,11 +40,6 @@ public class RegexpQueryParser implements QueryParser<RegexpQueryBuilder> { public static final ParseField VALUE_FIELD = new ParseField("value"); @Override - public String[] names() { - return new String[]{RegexpQueryBuilder.NAME}; - } - - @Override public RegexpQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -83,7 +80,8 @@ public class RegexpQueryParser implements QueryParser<RegexpQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[regexp] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[regexp] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/ScriptQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/ScriptQueryParser.java index 51e299815b..fafd76e66f 100644 --- a/core/src/main/java/org/elasticsearch/index/query/ScriptQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/ScriptQueryParser.java @@ -36,14 +36,10 @@ import java.util.Map; */ public class ScriptQueryParser implements QueryParser<ScriptQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(ScriptQueryBuilder.NAME); public static final ParseField PARAMS_FIELD = new ParseField("params"); @Override - public String[] names() { - return new String[]{ScriptQueryBuilder.NAME}; - } - - @Override public ScriptQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); ScriptParameterParser scriptParameterParser = new ScriptParameterParser(); @@ -65,7 +61,8 @@ public class ScriptQueryParser implements QueryParser<ScriptQueryBuilder> { } else if (token == XContentParser.Token.START_OBJECT) { if (parseContext.parseFieldMatcher().match(currentFieldName, ScriptField.SCRIPT)) { script = Script.parse(parser, parseContext.parseFieldMatcher()); - } else if (parseContext.parseFieldMatcher().match(currentFieldName, PARAMS_FIELD)) { // TODO remove in 3.0 (here to support old script APIs) + } else if (parseContext.parseFieldMatcher().match(currentFieldName, PARAMS_FIELD)) { + // TODO remove in 3.0 (here to support old script APIs) params = parser.map(); } else { throw new ParsingException(parser.getTokenLocation(), "[script] query does not support [" + currentFieldName + "]"); @@ -90,7 +87,8 @@ public class ScriptQueryParser implements QueryParser<ScriptQueryBuilder> { script = new Script(scriptValue.script(), scriptValue.scriptType(), scriptParameterParser.lang(), params); } } else if (params != null) { - throw new ParsingException(parser.getTokenLocation(), "script params must be specified inside script object in a [script] filter"); + throw new ParsingException(parser.getTokenLocation(), + "script params must be specified inside script object in a [script] filter"); } if (script == null) { diff --git a/core/src/main/java/org/elasticsearch/index/query/SimpleQueryStringParser.java b/core/src/main/java/org/elasticsearch/index/query/SimpleQueryStringParser.java index 9fb5891935..3b29ef5ffe 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SimpleQueryStringParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SimpleQueryStringParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -61,6 +60,7 @@ import java.util.Map; */ public class SimpleQueryStringParser implements QueryParser<SimpleQueryStringBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SimpleQueryStringBuilder.NAME); public static final ParseField MINIMUM_SHOULD_MATCH_FIELD = new ParseField("minimum_should_match"); public static final ParseField ANALYZE_WILDCARD_FIELD = new ParseField("analyze_wildcard"); public static final ParseField LENIENT_FIELD = new ParseField("lenient"); @@ -73,11 +73,6 @@ public class SimpleQueryStringParser implements QueryParser<SimpleQueryStringBui public static final ParseField FIELDS_FIELD = new ParseField("fields"); @Override - public String[] names() { - return new String[]{SimpleQueryStringBuilder.NAME, Strings.toCamelCase(SimpleQueryStringBuilder.NAME)}; - } - - @Override public SimpleQueryStringBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -120,7 +115,8 @@ public class SimpleQueryStringParser implements QueryParser<SimpleQueryStringBui fieldsAndWeights.put(fField, fBoost); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + SimpleQueryStringBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + SimpleQueryStringBuilder.NAME + + "] query does not support [" + currentFieldName + "]"); } } else if (token.isValue()) { if (parseContext.parseFieldMatcher().match(currentFieldName, QUERY_FIELD)) { @@ -156,10 +152,12 @@ public class SimpleQueryStringParser implements QueryParser<SimpleQueryStringBui } else if (parseContext.parseFieldMatcher().match(currentFieldName, MINIMUM_SHOULD_MATCH_FIELD)) { minimumShouldMatch = parser.textOrNull(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + SimpleQueryStringBuilder.NAME + "] unsupported field [" + parser.currentName() + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + SimpleQueryStringBuilder.NAME + + "] unsupported field [" + parser.currentName() + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + SimpleQueryStringBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), "[" + SimpleQueryStringBuilder.NAME + + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryBuilder.java index 6f2f40f22d..25f061a533 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryBuilder.java @@ -32,12 +32,14 @@ import java.util.Objects; /** * Builder for {@link org.apache.lucene.search.spans.SpanContainingQuery}. */ -public class SpanContainingQueryBuilder extends AbstractQueryBuilder<SpanContainingQueryBuilder> implements SpanQueryBuilder<SpanContainingQueryBuilder> { +public class SpanContainingQueryBuilder extends AbstractQueryBuilder<SpanContainingQueryBuilder> + implements SpanQueryBuilder<SpanContainingQueryBuilder> { public static final String NAME = "span_containing"; private final SpanQueryBuilder big; private final SpanQueryBuilder little; - static final SpanContainingQueryBuilder PROTOTYPE = new SpanContainingQueryBuilder(SpanTermQueryBuilder.PROTOTYPE, SpanTermQueryBuilder.PROTOTYPE); + static final SpanContainingQueryBuilder PROTOTYPE = + new SpanContainingQueryBuilder(SpanTermQueryBuilder.PROTOTYPE, SpanTermQueryBuilder.PROTOTYPE); /** * @param big the big clause, it must enclose {@code little} for a match. diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryParser.java index c9404695a4..53857aaba8 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanContainingQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,15 +30,11 @@ import java.io.IOException; */ public class SpanContainingQueryParser implements QueryParser<SpanContainingQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanContainingQueryBuilder.NAME); public static final ParseField BIG_FIELD = new ParseField("big"); public static final ParseField LITTLE_FIELD = new ParseField("little"); @Override - public String[] names() { - return new String[]{SpanContainingQueryBuilder.NAME, Strings.toCamelCase(SpanContainingQueryBuilder.NAME)}; - } - - @Override public SpanContainingQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); float boost = AbstractQueryBuilder.DEFAULT_BOOST; @@ -66,14 +61,16 @@ public class SpanContainingQueryParser implements QueryParser<SpanContainingQuer } little = (SpanQueryBuilder<?>) query; } else { - throw new ParsingException(parser.getTokenLocation(), "[span_containing] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[span_containing] query does not support [" + currentFieldName + "]"); } } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[span_containing] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[span_containing] query does not support [" + currentFieldName + "]"); } } diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanFirstQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanFirstQueryParser.java index 7d466dd4ce..4465af39a4 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanFirstQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanFirstQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,15 +30,11 @@ import java.io.IOException; */ public class SpanFirstQueryParser implements QueryParser<SpanFirstQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanFirstQueryBuilder.NAME); public static final ParseField MATCH_FIELD = new ParseField("match"); public static final ParseField END_FIELD = new ParseField("end"); @Override - public String[] names() { - return new String[]{SpanFirstQueryBuilder.NAME, Strings.toCamelCase(SpanFirstQueryBuilder.NAME)}; - } - - @Override public SpanFirstQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryBuilder.java index 21c9c61555..40e4d20fb8 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryBuilder.java @@ -35,7 +35,8 @@ import java.util.Objects; * Query that allows wraping a {@link MultiTermQueryBuilder} (one of wildcard, fuzzy, prefix, term, range or regexp query) * as a {@link SpanQueryBuilder} so it can be nested. */ -public class SpanMultiTermQueryBuilder extends AbstractQueryBuilder<SpanMultiTermQueryBuilder> implements SpanQueryBuilder<SpanMultiTermQueryBuilder> { +public class SpanMultiTermQueryBuilder extends AbstractQueryBuilder<SpanMultiTermQueryBuilder> + implements SpanQueryBuilder<SpanMultiTermQueryBuilder> { public static final String NAME = "span_multi"; private final MultiTermQueryBuilder multiTermQueryBuilder; diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryParser.java index a3b1c09b14..79f2d69904 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanMultiTermQueryParser.java @@ -20,7 +20,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -30,14 +29,10 @@ import java.io.IOException; */ public class SpanMultiTermQueryParser implements QueryParser<SpanMultiTermQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanMultiTermQueryBuilder.NAME); public static final ParseField MATCH_FIELD = new ParseField("match"); @Override - public String[] names() { - return new String[]{SpanMultiTermQueryBuilder.NAME, Strings.toCamelCase(SpanMultiTermQueryBuilder.NAME)}; - } - - @Override public SpanMultiTermQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); String currentFieldName = null; @@ -52,7 +47,8 @@ public class SpanMultiTermQueryParser implements QueryParser<SpanMultiTermQueryB if (parseContext.parseFieldMatcher().match(currentFieldName, MATCH_FIELD)) { QueryBuilder innerQuery = parseContext.parseInnerQueryBuilder(); if (innerQuery instanceof MultiTermQueryBuilder == false) { - throw new ParsingException(parser.getTokenLocation(), "[span_multi] [" + MATCH_FIELD.getPreferredName() + "] must be of type multi term query"); + throw new ParsingException(parser.getTokenLocation(), + "[span_multi] [" + MATCH_FIELD.getPreferredName() + "] must be of type multi term query"); } subQuery = (MultiTermQueryBuilder) innerQuery; } else { @@ -70,7 +66,8 @@ public class SpanMultiTermQueryParser implements QueryParser<SpanMultiTermQueryB } if (subQuery == null) { - throw new ParsingException(parser.getTokenLocation(), "[span_multi] must have [" + MATCH_FIELD.getPreferredName() + "] multi term query clause"); + throw new ParsingException(parser.getTokenLocation(), + "[span_multi] must have [" + MATCH_FIELD.getPreferredName() + "] multi term query clause"); } return new SpanMultiTermQueryBuilder(subQuery).queryName(queryName).boost(boost); diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryParser.java index fc899ac5d1..c00a44e214 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -33,17 +32,13 @@ import java.util.List; */ public class SpanNearQueryParser implements QueryParser<SpanNearQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanNearQueryBuilder.NAME); public static final ParseField SLOP_FIELD = new ParseField("slop"); public static final ParseField COLLECT_PAYLOADS_FIELD = new ParseField("collect_payloads").withAllDeprecated("no longer supported"); public static final ParseField CLAUSES_FIELD = new ParseField("clauses"); public static final ParseField IN_ORDER_FIELD = new ParseField("in_order"); @Override - public String[] names() { - return new String[]{SpanNearQueryBuilder.NAME, Strings.toCamelCase(SpanNearQueryBuilder.NAME)}; - } - - @Override public SpanNearQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanNotQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanNotQueryParser.java index f3dc8e008c..0da7938aab 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanNotQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanNotQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,6 +30,7 @@ import java.io.IOException; */ public class SpanNotQueryParser implements QueryParser<SpanNotQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanNotQueryBuilder.NAME); public static final ParseField POST_FIELD = new ParseField("post"); public static final ParseField PRE_FIELD = new ParseField("pre"); public static final ParseField DIST_FIELD = new ParseField("dist"); @@ -38,11 +38,6 @@ public class SpanNotQueryParser implements QueryParser<SpanNotQueryBuilder> { public static final ParseField INCLUDE_FIELD = new ParseField("include"); @Override - public String[] names() { - return new String[]{SpanNotQueryBuilder.NAME, Strings.toCamelCase(SpanNotQueryBuilder.NAME)}; - } - - @Override public SpanNotQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanOrQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanOrQueryParser.java index 50500def86..7c199fc857 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanOrQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanOrQueryParser.java @@ -33,14 +33,10 @@ import java.util.List; */ public class SpanOrQueryParser implements QueryParser<SpanOrQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanOrQueryBuilder.NAME); public static final ParseField CLAUSES_FIELD = new ParseField("clauses"); @Override - public String[] names() { - return new String[]{SpanOrQueryBuilder.NAME, Strings.toCamelCase(SpanOrQueryBuilder.NAME)}; - } - - @Override public SpanOrQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanTermQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanTermQueryParser.java index 930600040f..254c82fca7 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanTermQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanTermQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,14 +30,10 @@ import java.io.IOException; */ public class SpanTermQueryParser implements QueryParser<SpanTermQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanTermQueryBuilder.NAME); public static final ParseField TERM_FIELD = new ParseField("term"); @Override - public String[] names() { - return new String[]{SpanTermQueryBuilder.NAME, Strings.toCamelCase(SpanTermQueryBuilder.NAME)}; - } - - @Override public SpanTermQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException, ParsingException { XContentParser parser = parseContext.parser(); @@ -70,7 +65,8 @@ public class SpanTermQueryParser implements QueryParser<SpanTermQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[span_term] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[span_term] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryBuilder.java index 440e1797df..40298debd3 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryBuilder.java @@ -32,12 +32,14 @@ import java.util.Objects; /** * Builder for {@link org.apache.lucene.search.spans.SpanWithinQuery}. */ -public class SpanWithinQueryBuilder extends AbstractQueryBuilder<SpanWithinQueryBuilder> implements SpanQueryBuilder<SpanWithinQueryBuilder> { +public class SpanWithinQueryBuilder extends AbstractQueryBuilder<SpanWithinQueryBuilder> + implements SpanQueryBuilder<SpanWithinQueryBuilder> { public static final String NAME = "span_within"; private final SpanQueryBuilder big; private final SpanQueryBuilder little; - static final SpanWithinQueryBuilder PROTOTYPE = new SpanWithinQueryBuilder(SpanTermQueryBuilder.PROTOTYPE, SpanTermQueryBuilder.PROTOTYPE); + static final SpanWithinQueryBuilder PROTOTYPE = + new SpanWithinQueryBuilder(SpanTermQueryBuilder.PROTOTYPE, SpanTermQueryBuilder.PROTOTYPE); /** * Query that returns spans from <code>little</code> that are contained in a spans from <code>big</code>. diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryParser.java index 16d4a9de82..194961ba36 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanWithinQueryParser.java @@ -21,7 +21,6 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; -import org.elasticsearch.common.Strings; import org.elasticsearch.common.xcontent.XContentParser; import java.io.IOException; @@ -31,15 +30,11 @@ import java.io.IOException; */ public class SpanWithinQueryParser implements QueryParser<SpanWithinQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(SpanWithinQueryBuilder.NAME); public static final ParseField BIG_FIELD = new ParseField("big"); public static final ParseField LITTLE_FIELD = new ParseField("little"); @Override - public String[] names() { - return new String[]{SpanWithinQueryBuilder.NAME, Strings.toCamelCase(SpanWithinQueryBuilder.NAME)}; - } - - @Override public SpanWithinQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -67,7 +62,8 @@ public class SpanWithinQueryParser implements QueryParser<SpanWithinQueryBuilder } little = (SpanQueryBuilder) query; } else { - throw new ParsingException(parser.getTokenLocation(), "[span_within] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[span_within] query does not support [" + currentFieldName + "]"); } } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); diff --git a/core/src/main/java/org/elasticsearch/index/query/TemplateQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/TemplateQueryParser.java index 841508506c..d6141ee8ef 100644 --- a/core/src/main/java/org/elasticsearch/index/query/TemplateQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/TemplateQueryParser.java @@ -19,6 +19,7 @@ package org.elasticsearch.index.query; import org.elasticsearch.common.Nullable; +import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParseFieldMatcher; import org.elasticsearch.common.xcontent.XContentParser; import org.elasticsearch.script.ScriptService; @@ -41,10 +42,7 @@ public class TemplateQueryParser implements QueryParser<TemplateQueryBuilder> { parametersToTypes.put("id", ScriptService.ScriptType.INDEXED); } - @Override - public String[] names() { - return new String[] {TemplateQueryBuilder.NAME}; - } + public static final ParseField QUERY_NAME_FIELD = new ParseField(TemplateQueryBuilder.NAME); /** * Parses the template query replacing template parameters with provided @@ -69,7 +67,8 @@ public class TemplateQueryParser implements QueryParser<TemplateQueryBuilder> { return parse(parser, parameterMap, parseFieldMatcher); } - public static Template parse(String defaultLang, XContentParser parser, ParseFieldMatcher parseFieldMatcher, String... parameters) throws IOException { + public static Template parse(String defaultLang, XContentParser parser, + ParseFieldMatcher parseFieldMatcher, String... parameters) throws IOException { Map<String, ScriptService.ScriptType> parameterMap = new HashMap<>(parametersToTypes); for (String parameter : parameters) { @@ -82,7 +81,8 @@ public class TemplateQueryParser implements QueryParser<TemplateQueryBuilder> { return parse(parser, parametersToTypes, parseFieldMatcher); } - public static Template parse(XContentParser parser, Map<String, ScriptService.ScriptType> parameterMap, ParseFieldMatcher parseFieldMatcher) throws IOException { + public static Template parse(XContentParser parser, Map<String, ScriptService.ScriptType> parameterMap, + ParseFieldMatcher parseFieldMatcher) throws IOException { return Template.parse(parser, parameterMap, parseFieldMatcher); } diff --git a/core/src/main/java/org/elasticsearch/index/query/TermQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/TermQueryParser.java index 5ac083d621..7ad9947561 100644 --- a/core/src/main/java/org/elasticsearch/index/query/TermQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/TermQueryParser.java @@ -30,15 +30,11 @@ import java.io.IOException; */ public class TermQueryParser implements QueryParser<TermQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(TermQueryBuilder.NAME); public static final ParseField TERM_FIELD = new ParseField("term"); public static final ParseField VALUE_FIELD = new ParseField("value"); @Override - public String[] names() { - return new String[]{TermQueryBuilder.NAME}; - } - - @Override public TermQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -56,7 +52,8 @@ public class TermQueryParser implements QueryParser<TermQueryBuilder> { } else if (token == XContentParser.Token.START_OBJECT) { // also support a format of "term" : {"field_name" : { ... }} if (fieldName != null) { - throw new ParsingException(parser.getTokenLocation(), "[term] query does not support different field names, use [bool] query instead"); + throw new ParsingException(parser.getTokenLocation(), + "[term] query does not support different field names, use [bool] query instead"); } fieldName = currentFieldName; while ((token = parser.nextToken()) != XContentParser.Token.END_OBJECT) { @@ -72,13 +69,15 @@ public class TermQueryParser implements QueryParser<TermQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.BOOST_FIELD)) { boost = parser.floatValue(); } else { - throw new ParsingException(parser.getTokenLocation(), "[term] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[term] query does not support [" + currentFieldName + "]"); } } } } else if (token.isValue()) { if (fieldName != null) { - throw new ParsingException(parser.getTokenLocation(), "[term] query does not support different field names, use [bool] query instead"); + throw new ParsingException(parser.getTokenLocation(), + "[term] query does not support different field names, use [bool] query instead"); } fieldName = currentFieldName; value = parser.objectBytes(); diff --git a/core/src/main/java/org/elasticsearch/index/query/TermsQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/TermsQueryParser.java index d92b87d3af..62ecc7807e 100644 --- a/core/src/main/java/org/elasticsearch/index/query/TermsQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/TermsQueryParser.java @@ -19,6 +19,7 @@ package org.elasticsearch.index.query; +import org.elasticsearch.common.ParseField; import org.elasticsearch.common.ParsingException; import org.elasticsearch.common.xcontent.XContentParser; import org.elasticsearch.indices.TermsLookup; @@ -37,10 +38,7 @@ import java.util.List; */ public class TermsQueryParser implements QueryParser { - @Override - public String[] names() { - return new String[]{TermsQueryBuilder.NAME, "in"}; - } + public static final ParseField QUERY_NAME_FIELD = new ParseField(TermsQueryBuilder.NAME, "in"); @Override public QueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { @@ -62,13 +60,15 @@ public class TermsQueryParser implements QueryParser { // skip } else if (token == XContentParser.Token.START_ARRAY) { if (fieldName != null) { - throw new ParsingException(parser.getTokenLocation(), "[" + TermsQueryBuilder.NAME + "] query does not support multiple fields"); + throw new ParsingException(parser.getTokenLocation(), + "[" + TermsQueryBuilder.NAME + "] query does not support multiple fields"); } fieldName = currentFieldName; values = parseValues(parser); } else if (token == XContentParser.Token.START_OBJECT) { if (fieldName != null) { - throw new ParsingException(parser.getTokenLocation(), "[" + TermsQueryBuilder.NAME + "] query does not support more than one field. " + throw new ParsingException(parser.getTokenLocation(), + "[" + TermsQueryBuilder.NAME + "] query does not support more than one field. " + "Already got: [" + fieldName + "] but also found [" + currentFieldName +"]"); } fieldName = currentFieldName; @@ -79,15 +79,18 @@ public class TermsQueryParser implements QueryParser { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + TermsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[" + TermsQueryBuilder.NAME + "] query does not support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + TermsQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[" + TermsQueryBuilder.NAME + "] unknown token [" + token + "] after [" + currentFieldName + "]"); } } if (fieldName == null) { - throw new ParsingException(parser.getTokenLocation(), "[" + TermsQueryBuilder.NAME + "] query requires a field name, followed by array of terms or a document lookup specification"); + throw new ParsingException(parser.getTokenLocation(), "[" + TermsQueryBuilder.NAME + "] query requires a field name, " + + "followed by array of terms or a document lookup specification"); } return new TermsQueryBuilder(fieldName, values, termsLookup) .boost(boost) diff --git a/core/src/main/java/org/elasticsearch/index/query/TypeQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/TypeQueryParser.java index d746b4656a..ee60317365 100644 --- a/core/src/main/java/org/elasticsearch/index/query/TypeQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/TypeQueryParser.java @@ -31,14 +31,10 @@ import java.io.IOException; */ public class TypeQueryParser implements QueryParser<TypeQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(TypeQueryBuilder.NAME); public static final ParseField VALUE_FIELD = new ParseField("value"); @Override - public String[] names() { - return new String[]{TypeQueryBuilder.NAME}; - } - - @Override public TypeQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); BytesRef type = null; @@ -59,15 +55,18 @@ public class TypeQueryParser implements QueryParser<TypeQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, VALUE_FIELD)) { type = parser.utf8Bytes(); } else { - throw new ParsingException(parser.getTokenLocation(), "[" + TypeQueryBuilder.NAME + "] filter doesn't support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[" + TypeQueryBuilder.NAME + "] filter doesn't support [" + currentFieldName + "]"); } } else { - throw new ParsingException(parser.getTokenLocation(), "[" + TypeQueryBuilder.NAME + "] filter doesn't support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[" + TypeQueryBuilder.NAME + "] filter doesn't support [" + currentFieldName + "]"); } } if (type == null) { - throw new ParsingException(parser.getTokenLocation(), "[" + TypeQueryBuilder.NAME + "] filter needs to be provided with a value for the type"); + throw new ParsingException(parser.getTokenLocation(), + "[" + TypeQueryBuilder.NAME + "] filter needs to be provided with a value for the type"); } return new TypeQueryBuilder(type) .boost(boost) diff --git a/core/src/main/java/org/elasticsearch/index/query/WildcardQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/WildcardQueryBuilder.java index 314bc6faeb..d14f227ed3 100644 --- a/core/src/main/java/org/elasticsearch/index/query/WildcardQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/WildcardQueryBuilder.java @@ -42,7 +42,8 @@ import java.util.Objects; * a Wildcard term should not start with one of the wildcards <tt>*</tt> or * <tt>?</tt>. */ -public class WildcardQueryBuilder extends AbstractQueryBuilder<WildcardQueryBuilder> implements MultiTermQueryBuilder<WildcardQueryBuilder> { +public class WildcardQueryBuilder extends AbstractQueryBuilder<WildcardQueryBuilder> + implements MultiTermQueryBuilder<WildcardQueryBuilder> { public static final String NAME = "wildcard"; diff --git a/core/src/main/java/org/elasticsearch/index/query/WildcardQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/WildcardQueryParser.java index e5482bc48c..0df16b7e63 100644 --- a/core/src/main/java/org/elasticsearch/index/query/WildcardQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/WildcardQueryParser.java @@ -30,16 +30,12 @@ import java.io.IOException; */ public class WildcardQueryParser implements QueryParser<WildcardQueryBuilder> { + public static final ParseField QUERY_NAME_FIELD = new ParseField(WildcardQueryBuilder.NAME); public static final ParseField WILDCARD_FIELD = new ParseField("wildcard"); public static final ParseField VALUE_FIELD = new ParseField("value"); public static final ParseField REWRITE_FIELD = new ParseField("rewrite"); @Override - public String[] names() { - return new String[]{WildcardQueryBuilder.NAME}; - } - - @Override public WildcardQueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); @@ -71,7 +67,8 @@ public class WildcardQueryParser implements QueryParser<WildcardQueryBuilder> { } else if (parseContext.parseFieldMatcher().match(currentFieldName, AbstractQueryBuilder.NAME_FIELD)) { queryName = parser.text(); } else { - throw new ParsingException(parser.getTokenLocation(), "[wildcard] query does not support [" + currentFieldName + "]"); + throw new ParsingException(parser.getTokenLocation(), + "[wildcard] query does not support [" + currentFieldName + "]"); } } } diff --git a/core/src/main/java/org/elasticsearch/index/query/WrapperQueryParser.java b/core/src/main/java/org/elasticsearch/index/query/WrapperQueryParser.java index a18ad52fcf..0190c3da3c 100644 --- a/core/src/main/java/org/elasticsearch/index/query/WrapperQueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/WrapperQueryParser.java @@ -30,14 +30,10 @@ import java.io.IOException; */ public class WrapperQueryParser implements QueryParser { + public static final ParseField QUERY_NAME_FIELD = new ParseField(WrapperQueryBuilder.NAME); public static final ParseField QUERY_FIELD = new ParseField("query"); @Override - public String[] names() { - return new String[]{WrapperQueryBuilder.NAME}; - } - - @Override public QueryBuilder fromXContent(QueryParseContext parseContext) throws IOException { XContentParser parser = parseContext.parser(); diff --git a/core/src/main/java/org/elasticsearch/index/query/functionscore/FunctionScoreQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/functionscore/FunctionScoreQueryBuilder.java index 5b3a2faccb..5faffd7f97 100644 --- a/core/src/main/java/org/elasticsearch/index/query/functionscore/FunctionScoreQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/functionscore/FunctionScoreQueryBuilder.java @@ -63,6 +63,8 @@ public class FunctionScoreQueryBuilder extends AbstractQueryBuilder<FunctionScor public static final String NAME = "function_score"; + public static final ParseField QUERY_NAME_FIELD = new ParseField(NAME); + // For better readability of error message static final String MISPLACED_FUNCTION_MESSAGE_PREFIX = "you can either define [functions] array or a single function, not both. "; |