Skip to content

Commit 0a93a03

Browse files
author
Christoph Büscher
authored
Remove deprecated FieldNamesFieldMapper.Builder#index (#37305)
The method calls "enabled" in addition to what the super.index() does, but this seems to be done explicitely now in the TypeParsers `parse` method. The removed method has been deprecated since at least 6.0. Also making some of the Builders methods and ctos private since they are only used internally in this class.
1 parent 757932a commit 0a93a03

File tree

1 file changed

+3
-10
lines changed

1 file changed

+3
-10
lines changed

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

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -73,21 +73,14 @@ public static class Defaults {
7373
}
7474
}
7575

76-
public static class Builder extends MetadataFieldMapper.Builder<Builder, FieldNamesFieldMapper> {
76+
private static class Builder extends MetadataFieldMapper.Builder<Builder, FieldNamesFieldMapper> {
7777
private boolean enabled = Defaults.ENABLED;
7878

79-
public Builder(MappedFieldType existing) {
79+
private Builder(MappedFieldType existing) {
8080
super(Defaults.NAME, existing == null ? Defaults.FIELD_TYPE : existing, Defaults.FIELD_TYPE);
8181
}
8282

83-
@Override
84-
@Deprecated
85-
public Builder index(boolean index) {
86-
enabled(index);
87-
return super.index(index);
88-
}
89-
90-
public Builder enabled(boolean enabled) {
83+
private Builder enabled(boolean enabled) {
9184
this.enabled = enabled;
9285
return this;
9386
}

0 commit comments

Comments
 (0)