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

TEST: Update settings should go through cluster state #29682

Merged
merged 4 commits into from Apr 26, 2018
Merged

TEST: Update settings should go through cluster state #29682

merged 4 commits into from Apr 26, 2018

Commits on Apr 24, 2018

  1. TEST: Update settings should go through cluster state 

    Today we update index settings directly via IndexService instead of the
cluster state in IndexServiceTests. However, those changes will be lost
if there is a cluster state update. In general, we should update index
settings via client and limit the direct usage in only special tests.

This commit replaces direct usages by the updateSettings api of client.

Closes elastic#24491
    @dnhatn
    dnhatn committed Apr 24, 2018
    Copy the full SHA
    0a28564 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into test/remove-direct-settings

    @dnhatn
    dnhatn committed Apr 24, 2018
    Copy the full SHA
    e714175 View commit details
    Browse the repository at this point in the history

Commits on Apr 25, 2018

  1. Merge branch 'master' into test/remove-direct-settings

    @dnhatn
    dnhatn committed Apr 25, 2018
    Copy the full SHA
    794a0a6 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into test/remove-direct-settings

    @dnhatn
    dnhatn committed Apr 25, 2018
    Copy the full SHA
    953be64 View commit details
    Browse the repository at this point in the history