-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
fast_refresh setting for stateless #96660
fast_refresh setting for stateless #96660
Conversation
Relates ES-6234
e45f4e5
to
f70e20e
Compare
Pinging @elastic/es-distributed (Team:Distributed) |
@Tim-Brooks should we also change the default refresh interval value here as well? |
I don't think that is necessarily straightforward to do. Maybe we can branch on node type but it is messy to make an index setting rely on a node setting. We should have this default be it's own discussion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Please add an actual PR description when merging including the fact that this setting is only allowed with stateless node types.
For example:
Closes: #155330 Adds `index.fast_refresh` to `.apm-custom-link` in order to ensure fast index refreshes on serverless (1 second periodic refreshes instead of 5 second which is the new default on serverless). This is pending on Elasticsearch adding support for `index.fast_refresh` (elastic/elasticsearch#96660) --------- Co-authored-by: miriam.aparicio <miriam.aparicio@gmail.com> Co-authored-by: Miriam <31922082+MiriamAparicio@users.noreply.github.com>
Introduces fast_refresh setting for indices, which is only allowed on stateless nodes.
Relates ES-6234