- Issue created by @marvil07
- Merge request !244#3530660: Introduce time-limit option for search-api:index command β (Merged) created by marvil07
- π΅πͺPeru marvil07
Over MR 244 the new option is introduced.
From the commit message:
This time limit is a best effort, not a exact time out.
Given the underlying code uses Batch API to run the indexing, a more exact bail out is not truly possible.
Instead, do check after each batch is processed, and act accordingly.Sample use:
drush search-api:index --time-limit=30 node_index
- Issue was unassigned.
- Status changed to Needs review
about 1 month ago 12:14pm 29 August 2025 - π¦πΉAustria drunken monkey Vienna, Austria
drunken monkey β made their first commit to this issueβs fork.
- π¦πΉAustria drunken monkey Vienna, Austria
Looks great, thanks a lot! And sorry for taking so long to reply.
I just had some minor nitpicks and made some code style changes, otherwise this looks good to go.
Feel free to review my changes and tell me what you think, otherwise Iβll just merge this in a week or so. - π΅πͺPeru marvil07
Looks great, thanks a lot! And sorry for taking so long to reply.
π Thanks for maintaining this project!
I just had some minor nitpicks and made some code style changes, otherwise this looks good to go.
Those changes look quite good, thanks!
- π§πͺBelgium kristiaanvandeneynde Antwerp, Belgium
Found a minor issue when passing a time limit of 0. The code seems to specifically allow for this, but then any time limit of 0 is converted into -1.
- π¦πΉAustria drunken monkey Vienna, Austria
@kristiaanvandeneynde: Thanks for catching that, youβre right.
Should be fixed now.