- Issue created by @klausi
- Merge request !26fix(cron): Always apply a batch size limit for predictable performance β (Merged) created by klausi
- π¦πΉAustria klausi π¦πΉ Vienna
Uploading stable patch file for composer patches.
- π¦πΉAustria klausi π¦πΉ Vienna
Not sure why the default batch limit is set to 10, that is way too low.
In Drupal 7 we use 500 for the variable search_api_saved_searches_load_max.
Should we update the default value on install to 500?
- π¦πΉAustria drunken monkey Vienna, Austria
Makes sense and looks good, thanks!
Merged.Feel free to open a separate issue for increasing the default limit. Youβre right, 10 does seem pretty low, though 500 seems a little high. Maybe we can get some input from others and land on a good compromise.
I think my thinking was probably to pick a low number so that it wonβt lead to any errors even when the checks take a bit longer each, because users running into problems because of the low value will usually have a larger site and know more what theyβre doing, hopefully. But if we can find a number that is higher but still pretty safe, that would of course be better. -
drunken monkey β
committed fa243ac0 on 8.x-1.x authored by
klausi β
Issue #3527038 by klausi, drunken monkey: Fixed problems when running...
-
drunken monkey β
committed fa243ac0 on 8.x-1.x authored by
klausi β