-
Notifications
You must be signed in to change notification settings - Fork 24.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix conflict when updating mapping with _all disabled #8426
Conversation
_all reports a conflict since elastic#7377. However, it was not checked if _all was actually configured in the updated mapping. Therefore whenever _all was disabled a mapping could not be updated unless _all was again added to the updated mapping. Also, add enabled setting to mapping always whenever enabled was set explicitely. closes elastic#8423
LGTM. |
@brwe want to get this in? |
_all reports a conflict since #7377. However, it was not checked if _all was actually configured in the updated mapping. Therefore whenever _all was disabled a mapping could not be updated unless _all was again added to the updated mapping. Also, add enabled setting to mapping always whenever enabled was set explicitely. closes #8423 closes #8426
_all reports a conflict since #7377. However, it was not checked if _all was actually configured in the updated mapping. Therefore whenever _all was disabled a mapping could not be updated unless _all was again added to the updated mapping. Also, add enabled setting to mapping always whenever enabled was set explicitely. closes #8423 closes #8426
It seems that it's not possible to disable the
Is this intentional? |
Hi @rayward Yes it is intentional. In fact from 2.0, the only time you'll be able to enable or disable the _all field is when you create the mapping. |
_all reports a conflict since elastic#7377. However, it was not checked if _all was actually configured in the updated mapping. Therefore whenever _all was disabled a mapping could not be updated unless _all was again added to the updated mapping. Also, add enabled setting to mapping always whenever enabled was set explicitely. closes elastic#8423 closes elastic#8426
_all reports a conflict since #7377. However, it was not checked if _all
was actually configured in the updated mapping. Therefore whenever _all
was disabled a mapping could not be updated unless _all was again added to the
updated mapping.
Also, add enabled setting to mapping always whenever enabled was set explicitely.
closes #8423