-
Notifications
You must be signed in to change notification settings - Fork 501
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 static setting for checkPendingFlushUpdate functionality of lucene index writer #6728
Conversation
13118b9
to
996c810
Compare
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.
Thank you, @CaptainDredge! Please provide the default for this setting.
_install-and-configure/configuring-opensearch/index-settings.md
Outdated
Show resolved
Hide resolved
…ene writer Signed-off-by: Prabhat Sharma <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]> Signed-off-by: Prabhat <[email protected]>
…ene writer Signed-off-by: Prabhat Sharma <[email protected]>
d3a44da
to
44fc4a0
Compare
Thanks @kolchfa-aws , I've added the default setting |
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. Thank you, @CaptainDredge!
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.
@kolchfa-aws @CaptainDredge Just a minor deletion. Otherwise, LGTM. Thanks!
_install-and-configure/configuring-opensearch/index-settings.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: kolchfa-aws <[email protected]>
Description
Does exactly what PR title says
Issues Resolved
Closes #6705
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.