Skip to content

Commit 8994262

Browse files
benwtrentelasticsearchmachine
andauthored
This fixes a bug for non-docvalue numerics introduced in elastic#128763 (elastic#130112)
* This fixes a bug for non-docvalue numerics introduced in elastic#128763 * [CI] Auto commit changes from spotless --------- Co-authored-by: elasticsearchmachine <[email protected]>
1 parent 671d906 commit 8994262

File tree

3 files changed

+3
-6
lines changed

3 files changed

+3
-6
lines changed

modules/mapper-extras/src/test/java/org/elasticsearch/index/mapper/extras/ScaledFloatFieldTypeTests.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
import org.elasticsearch.index.mapper.MappedFieldType;
3131
import org.elasticsearch.index.mapper.MapperBuilderContext;
3232
import org.elasticsearch.index.mapper.NumberFieldMapper;
33-
import org.elasticsearch.lucene.document.NumericField;
3433

3534
import java.io.IOException;
3635
import java.util.Arrays;
@@ -48,7 +47,7 @@ public void testTermQuery() {
4847
);
4948
double value = (randomDouble() * 2 - 1) * 10000;
5049
long scaledValue = Math.round(value * ft.getScalingFactor());
51-
assertEquals(NumericField.newExactLongQuery("scaled_float", scaledValue), ft.termQuery(value, MOCK_CONTEXT));
50+
assertEquals(LongPoint.newExactQuery("scaled_float", scaledValue), ft.termQuery(value, MOCK_CONTEXT));
5251

5352
MappedFieldType ft2 = new ScaledFloatFieldMapper.ScaledFloatFieldType("scaled_float", 0.1 + randomDouble() * 100, false);
5453
ElasticsearchException e2 = expectThrows(ElasticsearchException.class, () -> ft2.termQuery("42", MOCK_CONTEXT_DISALLOW_EXPENSIVE));

server/src/main/java/org/elasticsearch/index/mapper/NumberFieldMapper.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@
4545
import org.elasticsearch.index.fielddata.plain.SortedNumericIndexFieldData;
4646
import org.elasticsearch.index.mapper.TimeSeriesParams.MetricType;
4747
import org.elasticsearch.index.query.SearchExecutionContext;
48-
import org.elasticsearch.lucene.document.NumericField;
4948
import org.elasticsearch.lucene.search.XIndexSortSortedNumericDocValuesRangeQuery;
5049
import org.elasticsearch.script.DoubleFieldScript;
5150
import org.elasticsearch.script.LongFieldScript;
@@ -1247,7 +1246,7 @@ public Query termQuery(String field, Object value, boolean isIndexed) {
12471246

12481247
long v = parse(value, true);
12491248
if (isIndexed) {
1250-
return NumericField.newExactLongQuery(field, v);
1249+
return LongPoint.newExactQuery(field, v);
12511250
} else {
12521251
return SortedNumericDocValuesField.newSlowExactQuery(field, v);
12531252
}

server/src/test/java/org/elasticsearch/index/mapper/NumberFieldTypeTests.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@
4444
import org.elasticsearch.index.mapper.NumberFieldMapper.NumberType;
4545
import org.elasticsearch.index.query.SearchExecutionContext;
4646
import org.elasticsearch.index.query.SearchExecutionContextHelper;
47-
import org.elasticsearch.lucene.document.NumericField;
4847
import org.elasticsearch.lucene.search.XIndexSortSortedNumericDocValuesRangeQuery;
4948
import org.elasticsearch.script.ScriptCompiler;
5049
import org.elasticsearch.search.MultiValueMode;
@@ -147,7 +146,7 @@ private static MappedFieldType unsearchable() {
147146

148147
public void testTermQuery() {
149148
MappedFieldType ft = new NumberFieldMapper.NumberFieldType("field", NumberFieldMapper.NumberType.LONG);
150-
assertEquals(NumericField.newExactLongQuery("field", 42), ft.termQuery("42", MOCK_CONTEXT));
149+
assertEquals(LongPoint.newExactQuery("field", 42), ft.termQuery("42", MOCK_CONTEXT));
151150

152151
ft = new NumberFieldMapper.NumberFieldType("field", NumberFieldMapper.NumberType.LONG, false);
153152
assertEquals(SortedNumericDocValuesField.newSlowExactQuery("field", 42), ft.termQuery("42", MOCK_CONTEXT));

0 commit comments

Comments
 (0)