-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[Feature Request] Migrate Update by query, delete by query, reindex to use PiT instead of scroll #12923
Comments
@shwetathareja If i can try it if you haven't do? I was planning to run those operations concurrently #14516, but it seems depend on the pr, Can I also pick up this? |
@kkewwei If we change the internal implementation, can you also wrap in a privileged block like is done on this PR? #17250 If the internals aren't wrapped, then users may need to update roles definitions when using the security plugin. As long as the user's have permission to use the respective action like Update by Query I think we shouldn't need to authorize any additional transport actions executed internally. |
@cwperks Get it, thanks. |
@shwetathareja @peternied @cwperks I understand that the advantages of |
Is your feature request related to a problem? Please describe
Today, update by query, delete by query and reindex uses scroll API for searching.
Describe the solution you'd like
We should move these APIs to using Point in Time instead of scroll. It is now the preferred pagination for OpenSearch.
Related component
Search
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: