Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Simplify dynamic mappings updates. #10720

Merged
merged 1 commit into from Apr 23, 2015
Merged

Simplify dynamic mappings updates. #10720

merged 1 commit into from Apr 23, 2015

Commits on Apr 23, 2015

  1. Mappings: simplify dynamic mappings updates. 

    While dynamic mappings updates are using the same code path as updates from the
API when applied on a data node since elastic#10593, they were still using a different
code path on the master node. This commit makes dynamic updates processed the
same way as updates from the API, which also seems to do a better way at
acknowledgements (I could not reproduce the ConcurrentDynamicTemplateTests
failure anymore). It also adds more checks, like for instance that indexing on
replicas should not trigger dynamic mapping updates since they should have been
handled on the primary before.

Close elastic#10720
    @jpountz
    jpountz committed Apr 23, 2015
    Copy the full SHA
    c6cdf77 View commit details
    Browse the repository at this point in the history