diff options
author | Martijn van Groningen <martijn.v.groningen@gmail.com> | 2017-06-29 23:25:05 +0200 |
---|---|---|
committer | Martijn van Groningen <martijn.v.groningen@gmail.com> | 2017-06-30 13:48:18 +0200 |
commit | c8da7f84a20b836671048749245dba3f2c95dbeb (patch) | |
tree | f519b7573beb27feda954d6f299a9b2a53e8a760 /core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java | |
parent | 1fee1045b960ed1a582ff84ddd844e8b11bdfaa5 (diff) |
WrapperQueryBuilder should also rewrite the parsed query.
Failing to do so can cause other errors later on during query execution.
For example if `WrapperQueryBuilder` wraps a `GeoShapeQueryBuilder` that fetches the shape from an index then it will skip the shape fetching
and fail later with the error that no shapes have been fetched.
Diffstat (limited to 'core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java')
-rw-r--r-- | core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java b/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java index 68d2d369bb..17cb4d0c4f 100644 --- a/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java +++ b/core/src/test/java/org/elasticsearch/index/query/BoolQueryBuilderTests.java @@ -409,7 +409,7 @@ public class BoolQueryBuilderTests extends AbstractQueryTestCase<BoolQueryBuilde boolQueryBuilder.must(new WrapperQueryBuilder(new WrapperQueryBuilder(new MatchAllQueryBuilder().toString()).toString())); QueryBuilder rewritten = boolQueryBuilder.rewrite(createShardContext()); BoolQueryBuilder expected = new BoolQueryBuilder(); - expected.must(new WrapperQueryBuilder(new MatchAllQueryBuilder().toString())); + expected.must(new MatchAllQueryBuilder()); assertEquals(expected, rewritten); expected = new BoolQueryBuilder(); |