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
Allow executing multiple periodic flushes while they are being made durable #102571
Conversation
…urable Relates ES-7200
Pinging @elastic/es-distributed (Team:Distributed) |
Hi @fcofdez, I've created a changelog YAML for you. |
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.
* Executes the given flush request against the engine. | ||
* | ||
* @param request the flush request | ||
* @param listener to notify after fully durability has been achieved. |
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:
* @param listener to notify after fully durability has been achieved. | |
* @param listener to notify after full durability has been achieved. |
} | ||
}); | ||
afterWriteOperation(); | ||
flushOrRollRunning.compareAndSet(true, false); |
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.
I think we have to still reset before calling the method back, since otherwise it will always bail out early on the re-invocation?
…urable (elastic#102571) Closes ES-7200
Relates ES-7200