-
Notifications
You must be signed in to change notification settings - Fork 25.8k
The enabled setting is not updatable (and probably shouldn't) #33566
Copy link
Copy link
Closed
Labels
:Search Foundations/MappingIndex mappings, including merging and defining field typesIndex mappings, including merging and defining field types>bugTeam:Search FoundationsMeta label for the Search Foundations team in ElasticsearchMeta label for the Search Foundations team in Elasticsearchhelp wantedadoptmeadoptme
Description
According to the docs , the enabled setting for mappings should be updatable (see TIP at the end), which it actually isn't:
PUT my_index
{
"mappings": {
"doc": {
"properties": {
"bla": {
"properties": {
"title": { "type": "text" }
}
}
}
}
}
}
GET /my_index/_mapping/doc
PUT /my_index/_mapping/doc
{
"properties": {
"bla": {
"enabled" : false
}
}
}
GET /my_index/_mapping/doc
-> returns same initial mapping :sadtrombone:
While implementing a fix, I noticed that we probably do not want to change and fix this, as this might lead to a surprising effects if the setting is changed during indexing (A field might be indexed on the primary but not on the replica (or vice versa)). Similarly, it might interfere in funny ways with reindexing.
I suggest updating the docs to say that this field is non-updateable, but wanted to raise it first for discussion with @elastic/es-search-aggs.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
:Search Foundations/MappingIndex mappings, including merging and defining field typesIndex mappings, including merging and defining field types>bugTeam:Search FoundationsMeta label for the Search Foundations team in ElasticsearchMeta label for the Search Foundations team in Elasticsearchhelp wantedadoptmeadoptme
Type
Fields
Give feedbackNo fields configured for issues without a type.