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

Indices.PutSettings can't specify index name in v8.11.0 #756

Closed
gannett-ggreer opened this issue Nov 16, 2023 · 2 comments
Closed

Indices.PutSettings can't specify index name in v8.11.0 #756

gannett-ggreer opened this issue Nov 16, 2023 · 2 comments
Assignees
Labels

Comments

@gannett-ggreer
Copy link

gannett-ggreer commented Nov 16, 2023

Possibly related to commit: 537d090

Previously the usage was: .client.Indices.PutSettings().Index(index).Request(settings).Do(ctx)

Now .Index() has changed to accept index *types.IndexSettings instead of index string and no method to set the index name exists.

There was a similar change to PutMappings in the same commit but there the index name can be set via the PutMappings(index) call after the .Index() method changed, which makes sense, but a similar change was not done for PutSettings().

@gannett-ggreer gannett-ggreer changed the title Indices.PutSettings can't specify index name after commit 537d09014a37fd1288919bd43eb7f1ab1bd374ae Indices.PutSettings can't specify index name in v8.11.0 Nov 16, 2023
@Anaethelion Anaethelion self-assigned this Nov 23, 2023
@Anaethelion
Copy link
Contributor

Thank you, this is indeed a bug in the generation process that validates collisions and priorities, I'll have a look.

@Anaethelion
Copy link
Contributor

Fixed in v8.11.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants