Skip to content
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.

Sign up for GitHub

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

Jump to bottom

scroll timeout cannot be changed in _reindex API #27555

Closed
jimczi opened this issue Nov 28, 2017 · 1 comment
Closed

scroll timeout cannot be changed in _reindex API #27555

jimczi opened this issue Nov 28, 2017 · 1 comment
Labels
:Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >enhancement

Comments

@jimczi
Copy link
Contributor

jimczi commented Nov 28, 2017

The scroll timeout is set by default to 5 minutes when using the _reindex API. It is possible to change this value when using the java client but not when using the rest API directly.
5 minutes is quite high but if the indexation is slow we can still reach the timeout and fail the entire reindex due to a context missing in the scroll. Since _update_by_query and _delete_by_query can change the default value in the rest API, we should also handle this parameter in _reindex.
@PnPie PnPie mentioned this issue Jan 1, 2018
Merged
@PnPie
Copy link
Contributor

PnPie commented Jan 1, 2018

Hello @jimczi
I made a PR for this 😄

@lcawl lcawl added :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. and removed :Reindex API labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@PnPie @jimczi @lcawl