diff --git a/core/src/main/java/org/elasticsearch/index/query/AbstractQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/AbstractQueryBuilder.java index 91adde07e4f..de184df8eb7 100644 --- a/core/src/main/java/org/elasticsearch/index/query/AbstractQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/AbstractQueryBuilder.java @@ -244,9 +244,6 @@ public abstract class AbstractQueryBuilder exte * {@link Query} instances. {@link QueryBuilder} that return null calling * their {@link QueryBuilder#toQuery(QueryShardContext)} method are not added to the * resulting collection. - * - * @throws IOException - * @throws QueryShardException */ protected static Collection toQueries(Collection queryBuilders, QueryShardContext context) throws QueryShardException, IOException { 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 ad238c629af..e382a9e7c6f 100644 --- a/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/GeoDistanceQueryBuilder.java @@ -292,10 +292,6 @@ public class GeoDistanceQueryBuilder extends AbstractQueryBuilderfalse. - * @param existance should the missing filter include documents where the field doesn't exist in the docs. + * @param existence should the missing filter include documents where the field doesn't exist in the docs. * Defaults to true. * @throws IllegalArgumentException when both existence and nullValue are set to false */ diff --git a/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryBuilder.java index 5e60a9a6307..7c5b15297b0 100644 --- a/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/MultiMatchQueryBuilder.java @@ -429,7 +429,7 @@ public class MultiMatchQueryBuilder extends AbstractQueryBuilder=1) representing the + * Set a cutoff value in [0..1] (or absolute number >=1) representing the * maximum threshold of a terms document frequency to be considered a low * frequency term. */ diff --git a/core/src/main/java/org/elasticsearch/index/query/QueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/QueryBuilder.java index e66c95eba75..3bc9bcb65d6 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryBuilder.java @@ -37,8 +37,6 @@ public interface QueryBuilder extends NamedWriteablenull if this query should be ignored upstream - * @throws QueryShardException - * @throws IOException */ Query toQuery(QueryShardContext context) throws IOException; @@ -49,8 +47,6 @@ public interface QueryBuilder extends NamedWriteablenull if this query should be ignored upstream - * @throws QueryShardException - * @throws IOException */ Query toFilter(QueryShardContext context) throws IOException; diff --git a/core/src/main/java/org/elasticsearch/index/query/QueryBuilders.java b/core/src/main/java/org/elasticsearch/index/query/QueryBuilders.java index d8d97aa6e38..42014a55b99 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryBuilders.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryBuilders.java @@ -617,7 +617,7 @@ public abstract class QueryBuilders { * A filter to filter based on a specific range from a specific geo location / point. * * @param name The location field name. - * @param point The point as geohash + * @param geohash The point as geohash */ public static GeoDistanceRangeQueryBuilder geoDistanceRangeQuery(String name, String geohash) { return new GeoDistanceRangeQueryBuilder(name, geohash); @@ -694,7 +694,6 @@ public abstract class QueryBuilders { * * @param name The shape field name * @param shape Shape to use in the filter - * @param relation relation of the shapes */ public static GeoShapeQueryBuilder geoShapeQuery(String name, ShapeBuilder shape) throws IOException { return new GeoShapeQueryBuilder(name, shape); @@ -769,7 +768,7 @@ public abstract class QueryBuilders { /** * A filter to filter only documents where a field does not exists in them. - * @param fieldPattern the field to query + * @param name the field to query */ public static MissingQueryBuilder missingQuery(String name) { return missingQuery(name, MissingQueryBuilder.DEFAULT_NULL_VALUE, MissingQueryBuilder.DEFAULT_EXISTENCE_VALUE); @@ -777,7 +776,7 @@ public abstract class QueryBuilders { /** * A filter to filter only documents where a field does not exists in them. - * @param fieldPattern the field to query + * @param name the field to query * @param nullValue should the missing filter automatically include fields with null value configured in the * mappings. Defaults to false. * @param existence should the missing filter include documents where the field doesn't exist in the docs. 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 7b44b3c24e8..2592e1d6dcb 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryParseContext.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryParseContext.java @@ -122,7 +122,6 @@ public class QueryParseContext { /** * @return a new QueryBuilder based on the current state of the parser - * @throws IOException */ public QueryBuilder parseInnerQueryBuilder() throws IOException { // move to START object @@ -163,7 +162,6 @@ public class QueryParseContext { /** * @return a new QueryBuilder based on the current state of the parser, but does so that the inner query * is parsed to a filter - * @throws IOException */ //norelease setting and checking the isFilter Flag should completely be moved to toQuery/toFilter after query refactoring public QueryBuilder parseInnerFilterToQueryBuilder() throws IOException { 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 564298fd321..56128b69cfd 100644 --- a/core/src/main/java/org/elasticsearch/index/query/QueryParser.java +++ b/core/src/main/java/org/elasticsearch/index/query/QueryParser.java @@ -39,7 +39,7 @@ public interface QueryParser> { * Parses the into a query from the current parser location. Will be at * "START_OBJECT" location, and should end when the token is at the matching * "END_OBJECT". - *

+ *

* Returns null if this query should be ignored in the context of * the DSL. */ @@ -56,7 +56,6 @@ public interface QueryParser> { * this context will be changed as a side effect of this method * call * @return the new QueryBuilder - * @throws IOException */ QB fromXContent(QueryParseContext parseContext) throws IOException; diff --git a/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryBuilder.java b/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryBuilder.java index b933cede248..a0435245653 100644 --- a/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryBuilder.java +++ b/core/src/main/java/org/elasticsearch/index/query/SpanNearQueryBuilder.java @@ -101,7 +101,7 @@ public class SpanNearQueryBuilder extends AbstractQueryBuilder> protected static String[] getCurrentTypes() { return currentTypes; } - + private static NamedWriteableRegistry namedWriteableRegistry; private static String[] randomTypes; @@ -140,7 +134,6 @@ public abstract class AbstractQueryTestCase> /** * Setup for the whole base test class. - * @throws IOException */ @BeforeClass public static void init() throws IOException { diff --git a/core/src/test/java/org/elasticsearch/index/query/DisMaxQueryBuilderTests.java b/core/src/test/java/org/elasticsearch/index/query/DisMaxQueryBuilderTests.java index 329a1623189..676634b526d 100644 --- a/core/src/test/java/org/elasticsearch/index/query/DisMaxQueryBuilderTests.java +++ b/core/src/test/java/org/elasticsearch/index/query/DisMaxQueryBuilderTests.java @@ -83,7 +83,6 @@ public class DisMaxQueryBuilderTests extends AbstractQueryTestCase /** * check that parser throws exception on missing values field - * @throws IOException */ @Test(expected=ParsingException.class) public void testIdsNotProvided() throws IOException { diff --git a/core/src/test/java/org/elasticsearch/index/query/NotQueryBuilderTests.java b/core/src/test/java/org/elasticsearch/index/query/NotQueryBuilderTests.java index b5baf4bb550..cae4ce5ebce 100644 --- a/core/src/test/java/org/elasticsearch/index/query/NotQueryBuilderTests.java +++ b/core/src/test/java/org/elasticsearch/index/query/NotQueryBuilderTests.java @@ -59,9 +59,6 @@ public class NotQueryBuilderTests extends AbstractQueryTestCase } } - /** - * @throws IOException - */ @Test(expected=ParsingException.class) public void testMissingFilterSection() throws IOException { String queryString = "{ \"not\" : {}"; diff --git a/core/src/test/java/org/elasticsearch/index/query/QueryFilterBuilderTests.java b/core/src/test/java/org/elasticsearch/index/query/QueryFilterBuilderTests.java index 8c9a815264a..15075b30921 100644 --- a/core/src/test/java/org/elasticsearch/index/query/QueryFilterBuilderTests.java +++ b/core/src/test/java/org/elasticsearch/index/query/QueryFilterBuilderTests.java @@ -54,7 +54,7 @@ public class QueryFilterBuilderTests extends AbstractQueryTestCasenull to pass on upwards + * test that wrapping an inner filter that returns null also returns null to pass on upwards */ @Test public void testInnerQueryReturnsNull() throws IOException {