Skip to content

Commit 2c65be2

Browse files
committed
better exception when a field is missing
1 parent cccefad commit 2c65be2

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,10 @@ public class RangeFilterParser extends AbstractIndexComponent implements XConten
103103
}
104104
}
105105

106+
if (fieldName == null) {
107+
throw new QueryParsingException(index, "No field specified for range filter");
108+
}
109+
106110
Filter filter = null;
107111
MapperService.SmartNameFieldMappers smartNameFieldMappers = parseContext.smartFieldMappers(fieldName);
108112
if (smartNameFieldMappers != null) {

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ public class TermFilterParser extends AbstractIndexComponent implements XContent
7575
}
7676
}
7777

78+
if (fieldName == null) {
79+
throw new QueryParsingException(index, "No field specified for term filter");
80+
}
81+
7882
if (value == null) {
7983
throw new QueryParsingException(index, "No value specified for term filter");
8084
}

0 commit comments

Comments
 (0)