Skip to content

Commit a0c5441

Browse files
getting alias fields for field caps
1 parent 53fc282 commit a0c5441

File tree

5 files changed

+12
-12
lines changed

5 files changed

+12
-12
lines changed

server/src/main/java/org/elasticsearch/action/fieldcaps/FieldCapabilitiesFetcher.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515
import org.elasticsearch.core.Nullable;
1616
import org.elasticsearch.index.IndexService;
1717
import org.elasticsearch.index.engine.Engine;
18-
import org.elasticsearch.index.mapper.FieldMapper;
1918
import org.elasticsearch.index.mapper.MappedFieldType;
19+
import org.elasticsearch.index.mapper.Mapper;
2020
import org.elasticsearch.index.mapper.RootObjectMapper;
2121
import org.elasticsearch.index.mapper.RuntimeField;
2222
import org.elasticsearch.index.query.MatchAllQueryBuilder;
@@ -170,16 +170,16 @@ static Map<String, IndexFieldCapabilities> retrieveFieldCaps(
170170
includeEmptyFields = includeEmptyFields || enableFieldHasValue == false;
171171
Map<String, IndexFieldCapabilities> responseMap = new HashMap<>();
172172

173-
// populate field mappers including metadata field mappers
173+
// populate with all types of mappers
174174
RootObjectMapper rootObjectMapper = context.getMappingLookup().getMapping().getRoot();
175-
List<FieldMapper> allMappers = new ArrayList<>();
175+
List<Mapper> allMappers = new ArrayList<>();
176176
allMappers.addAll(rootObjectMapper.getSourceFields());
177177
allMappers.addAll(context.getMetadataFields());
178178

179-
for (FieldMapper mapper : allMappers) {
179+
for (Mapper mapper : allMappers) {
180180
addFieldToFieldCaps(
181181
mapper.fullPath(),
182-
mapper.fieldType(),
182+
context.getFieldType(mapper.fullPath()),
183183
fieldNameFilter,
184184
fieldPredicate,
185185
filter,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,8 +121,8 @@ public int getTotalFieldsCount() {
121121
}
122122

123123
@Override
124-
public List<FieldMapper> getSourceFields() {
125-
return List.of();
124+
public List<Mapper> getSourceFields() {
125+
return List.of(this);
126126
}
127127

128128
public static class TypeParser implements Mapper.TypeParser {

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -450,8 +450,8 @@ public int getTotalFieldsCount() {
450450
}
451451

452452
@Override
453-
public List<FieldMapper> getSourceFields() {
454-
List<FieldMapper> fields = new ArrayList<>();
453+
public List<Mapper> getSourceFields() {
454+
List<Mapper> fields = new ArrayList<>();
455455
for (FieldMapper mapper : builderParams.multiFields.mappers) {
456456
fields.addAll(mapper.getSourceFields());
457457
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,5 +229,5 @@ public String getOffsetFieldName() {
229229
return null;
230230
}
231231

232-
public abstract List<FieldMapper> getSourceFields();
232+
public abstract List<Mapper> getSourceFields();
233233
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -269,8 +269,8 @@ public int getTotalFieldsCount() {
269269
}
270270

271271
@Override
272-
public List<FieldMapper> getSourceFields() {
273-
List<FieldMapper> fields = new ArrayList<>();
272+
public List<Mapper> getSourceFields() {
273+
List<Mapper> fields = new ArrayList<>();
274274
for (Mapper mapper : mappers.values()) {
275275
fields.addAll(mapper.getSourceFields());
276276
}

0 commit comments

Comments
 (0)