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.

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

Add support for is_write_index #3686

Closed
octocat-mona opened this issue Apr 18, 2019 · 2 comments
Closed

Add support for is_write_index #3686

octocat-mona opened this issue Apr 18, 2019 · 2 comments

Comments

@octocat-mona
Copy link

Describe the feature:

Allow setting of is_write_index for Create Index and Index Aliases API's which can be used by the Rollover API.

This seems currently not supported:

await client.CreateIndexAsync("my-index", s => s
    .Aliases(a => a
        .Alias("my-alias", d => d
            .IsWriteIndex() // <-- Method doesn't exist, only Filter() etc
        )
    )
);

Call via ES API:

PUT my-index
{
  "aliases": {
    "my-alias": {
      "is_write_index": true
    }
  }
}
russcam added a commit that referenced this issue Apr 29, 2019
This commit adds the ability to add an alias with is_write_index when creating an index using the Create Index API.

Closes #3686
@russcam
Copy link
Contributor

russcam commented Apr 29, 2019

Thanks for opening @KPStolk. Alias operations already had an ability to set is_write_index, but this was not exposed on aliases for the create index API. I have opened #3704 to address

russcam added a commit that referenced this issue May 7, 2019
This commit adds the ability to add an alias with is_write_index when creating an index using the Create Index API.

Closes #3686
russcam added a commit that referenced this issue May 7, 2019
This commit adds the ability to add an alias with is_write_index when creating an index using the Create Index API.

Closes #3686
@russcam
Copy link
Contributor

russcam commented May 8, 2019

Closing this as #3704 has been merged and will be in the next release

@russcam russcam closed this as completed May 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants