-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
Revisit indices options and make default values consistent #9438
Comments
+1 |
See #14102 |
One thing that recently came up is we should make a clearer distinction between internal options (e.g. forbidCloseIndices and ignoreAliases) and external ones (settable from REST e.g. ignore_unavailable, allow_no_indices and expand_wildcards). Unfortunately the internal ones are currently settable through java API while they should be unmodifiable (either one api can work against closed indices or it can not, it is a characteristics of the API itself). Also, I would consider unifying allow_no_indices and ignore_unavailable, my head starts spinning anytime I have to remember what the subtle difference between the two is. |
Pinging @elastic/es-core-infra |
This has been open for quite a while, and hasn't had a lot of interest. For now I'm going to close this as something we aren't planning on implementing. We can re-open it later if needed. |
We currently support different indices options within apis that execute against multiple indices: http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/multi-index.html .
We should revisit these indices options and defaults for each api and try to make them less confusing and more consistent.
Also, the code that applies these options, more specifically
MetaData#concreteIndices
, could use some love :)The text was updated successfully, but these errors were encountered: