Skip to content

Commit d43f71f

Browse files
committed
fix
1 parent b862d03 commit d43f71f

File tree

7 files changed

+16
-14
lines changed

7 files changed

+16
-14
lines changed

server/src/test/java/org/elasticsearch/index/mapper/blockloader/BooleanFieldBlockLoaderTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818

1919
public class BooleanFieldBlockLoaderTests extends BlockLoaderTestCase {
2020
public BooleanFieldBlockLoaderTests(Params params) {
21-
super(FieldType.BOOLEAN, params);
21+
super(FieldType.BOOLEAN.toString(), params);
2222
}
2323

2424
@Override

server/src/test/java/org/elasticsearch/index/mapper/blockloader/DateFieldBlockLoaderTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424

2525
public class DateFieldBlockLoaderTests extends BlockLoaderTestCase {
2626
public DateFieldBlockLoaderTests(Params params) {
27-
super(FieldType.DATE, params);
27+
super(FieldType.DATE.toString(), params);
2828
}
2929

3030
@Override

server/src/test/java/org/elasticsearch/index/mapper/blockloader/KeywordFieldBlockLoaderTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222

2323
public class KeywordFieldBlockLoaderTests extends BlockLoaderTestCase {
2424
public KeywordFieldBlockLoaderTests(Params params) {
25-
super(FieldType.KEYWORD, params);
25+
super(FieldType.KEYWORD.toString(), params);
2626
}
2727

2828
@SuppressWarnings("unchecked")

test/framework/src/main/java/org/elasticsearch/logsdb/datageneration/matchers/source/MappingTransforms.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ record FieldMapping(Map<String, Object> mappingParameters, List<Map<String, Obje
2626

2727
/**
2828
* Normalize mapping to have the same structure as normalized source and enable field mapping lookup.
29-
* Similar to {@link SourceTransforms#normalize(Map)} but needs to get rid of intermediate nodes
29+
* Similar to {@link SourceTransforms#normalize(Map, Map)} but needs to get rid of intermediate nodes
3030
* and collect results into a different data structure.
3131
*
3232
* @param map raw mapping document converted to map

test/framework/src/test/java/org/elasticsearch/logsdb/datageneration/DataGenerationSnapshotTests.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -220,11 +220,11 @@ public DataSourceResponse.FieldTypeGenerator handle(DataSourceRequest.FieldTypeG
220220
return new DataSourceResponse.FieldTypeGenerator(() -> {
221221
if (fieldType == FieldType.KEYWORD) {
222222
fieldType = FieldType.LONG;
223-
return new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(FieldType.KEYWORD);
223+
return new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(FieldType.KEYWORD.toString());
224224
}
225225

226226
fieldType = FieldType.KEYWORD;
227-
return new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(FieldType.LONG);
227+
return new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(FieldType.LONG.toString());
228228
});
229229
}
230230

@@ -235,11 +235,11 @@ public DataSourceResponse.DynamicMappingGenerator handle(DataSourceRequest.Dynam
235235

236236
@Override
237237
public DataSourceResponse.LeafMappingParametersGenerator handle(DataSourceRequest.LeafMappingParametersGenerator request) {
238-
if (request.fieldType() == FieldType.KEYWORD) {
238+
if (request.fieldType().equals(FieldType.KEYWORD.toString())) {
239239
return new DataSourceResponse.LeafMappingParametersGenerator(() -> Map.of("store", "true"));
240240
}
241241

242-
if (request.fieldType() == FieldType.LONG) {
242+
if (request.fieldType().equals(FieldType.LONG.toString())) {
243243
return new DataSourceResponse.LeafMappingParametersGenerator(() -> Map.of("index", "false"));
244244
}
245245

test/framework/src/test/java/org/elasticsearch/logsdb/datageneration/DataGenerationTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public DataSourceResponse.ChildFieldGenerator handle(DataSourceRequest.ChildFiel
9191
public DataSourceResponse.FieldTypeGenerator handle(DataSourceRequest.FieldTypeGenerator request) {
9292
return new DataSourceResponse.FieldTypeGenerator(
9393
() -> new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(
94-
FieldType.values()[generatedFields++ % FieldType.values().length]
94+
FieldType.values()[generatedFields++ % FieldType.values().length].toString()
9595
)
9696
);
9797

@@ -166,7 +166,7 @@ public DataSourceResponse.ObjectArrayGenerator handle(DataSourceRequest.ObjectAr
166166
@Override
167167
public DataSourceResponse.FieldTypeGenerator handle(DataSourceRequest.FieldTypeGenerator request) {
168168
return new DataSourceResponse.FieldTypeGenerator(
169-
() -> new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(FieldType.LONG)
169+
() -> new DataSourceResponse.FieldTypeGenerator.FieldTypeInfo(FieldType.LONG.toString())
170170
);
171171
}
172172
};

test/framework/src/test/java/org/elasticsearch/logsdb/datageneration/SourceMatcherTests.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ public void testDynamicMatch() throws IOException {
2727
);
2828

2929
var sut = new SourceMatcher(
30+
Map.of(),
3031
XContentBuilder.builder(XContentType.JSON.xContent()).startObject().endObject(),
3132
Settings.builder(),
3233
XContentBuilder.builder(XContentType.JSON.xContent()).startObject().endObject(),
@@ -49,6 +50,7 @@ public void testDynamicMismatch() throws IOException {
4950
);
5051

5152
var sut = new SourceMatcher(
53+
Map.of(),
5254
XContentBuilder.builder(XContentType.JSON.xContent()).startObject().endObject(),
5355
Settings.builder(),
5456
XContentBuilder.builder(XContentType.JSON.xContent()).startObject().endObject(),
@@ -77,7 +79,7 @@ public void testMappedMatch() throws IOException {
7779
mapping.endObject();
7880
mapping.endObject();
7981

80-
var sut = new SourceMatcher(mapping, Settings.builder(), mapping, Settings.builder(), values, values, false);
82+
var sut = new SourceMatcher(Map.of(), mapping, Settings.builder(), mapping, Settings.builder(), values, values, false);
8183
assertTrue(sut.match().isMatch());
8284
}
8385

@@ -102,7 +104,7 @@ public void testMappedMismatch() throws IOException {
102104
mapping.endObject();
103105
mapping.endObject();
104106

105-
var sut = new SourceMatcher(mapping, Settings.builder(), mapping, Settings.builder(), actual, expected, false);
107+
var sut = new SourceMatcher(Map.of(), mapping, Settings.builder(), mapping, Settings.builder(), actual, expected, false);
106108
assertFalse(sut.match().isMatch());
107109
}
108110

@@ -119,7 +121,7 @@ public void testCountedKeywordMatch() throws IOException {
119121
mapping.endObject();
120122
mapping.endObject();
121123

122-
var sut = new SourceMatcher(mapping, Settings.builder(), mapping, Settings.builder(), actual, expected, false);
124+
var sut = new SourceMatcher(Map.of(), mapping, Settings.builder(), mapping, Settings.builder(), actual, expected, false);
123125
assertTrue(sut.match().isMatch());
124126
}
125127

@@ -136,7 +138,7 @@ public void testCountedKeywordMismatch() throws IOException {
136138
mapping.endObject();
137139
mapping.endObject();
138140

139-
var sut = new SourceMatcher(mapping, Settings.builder(), mapping, Settings.builder(), actual, expected, false);
141+
var sut = new SourceMatcher(Map.of(), mapping, Settings.builder(), mapping, Settings.builder(), actual, expected, false);
140142
assertFalse(sut.match().isMatch());
141143
}
142144
}

0 commit comments

Comments
 (0)