-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[FEATURE] Support update index settings API #7703
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
Comments
Thanks for raising this, @SandervWickeren. Index settings are a little complex since we must explicitly distinguish between |
@stevejgordon thanks for the quick reply! In that case, I'll continue to use the NEST client alongside the new ElasticsearchClient for now, and port as soon as this feature is released. |
Hello, this is blocking for us too. Any plan to advance on that? |
Hi @TheFireCookie! The progress can be tracked in issue that was mentioned by Steve: As you can see, there is some stuff going on, but unfortunately not all prerequisites are done yet. |
Hi, fyi: @SandervWickeren , @TheFireCookie this functionality is implemented and I can confirm it works correctly thanks to @flobernd :) |
There is still a small part missing. The "unset" aka semantic null functionality is not implemented yet. I plan to ship this feature soon, but most of the Index Settings API is already functional 🙂 |
Elastic.Clients.Elasticsearch version: 8.1.0
Elasticsearch version: 8.7.x
.NET runtime version: .Net7
Operating system version: not applicable
Description of the problem including expected versus actual behavior:
As mentioned in Tune Index for Speed, you should update the index settings before and after indexing if you want to tune for speed. We are in a process of upgrading our Elastic setup from Nest to ElasticsearchClient and it seems like the update index settings api isn't supported yet.
Any chance this can be added in a next release?
Steps to reproduce:
UpdateSettingsAsync
, while it does exists for Nest.Expected behavior
Support added
The text was updated successfully, but these errors were encountered: