-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Add a feature flag for reindex resilience #138618
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 a feature flag for reindex resilience #138618
Conversation
This is not used yet, but several related streams of work will be hidden behind it until they are ready for production.
|
Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing) |
| /** | ||
| * A {@link FeatureFlag} to guard the work to make reindex more resilient while it is under development. | ||
| */ | ||
| static final FeatureFlag REINDEX_RESILIENCE_FEATURE_FLAG = new FeatureFlag("reindex_resilience"); |
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.
nit: I have seem usage like this which looks neat.
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.
That's a good idea. We're never going to do anything other than call isEnabled() on it, so it makes the call sites nicer to just be able to reference the boolean. So I've done that, please LMK how it looks to you.
samxbr
left a comment
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
This is not used yet, but several related streams of work will be hidden behind it until they are ready for production.
Closes elastic/elasticsearch-team#2084