Summary: | BatchDeleteBiblio task does not deal with indexation correctly | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | dcook, julian.maurice, kyle, martin.renvoize, tomascohen |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.05.00
|
|
Circulation function: | |||
Bug Depends on: | 27344 | ||
Bug Blocks: | 27341, 32366 | ||
Attachments: |
Bug 30460: Make BatchDeleteBiblio update the index in one request
Bug 30460: Make BatchDeleteBiblio update the index in one request Bug 30460: Make BatchDeleteBiblio update the index in one request |
Description
Jonathan Druart
2022-04-06 09:29:34 UTC
Created attachment 132989 [details] [review] Bug 30460: Make BatchDeleteBiblio update the index in one request No need to send 1 indexation request per item + 1 per biblio. Test plan: Confirm that the batch biblio deletion tool still works correctly Created attachment 134402 [details] [review] Bug 30460: Make BatchDeleteBiblio update the index in one request No need to send 1 indexation request per item + 1 per biblio. Test plan: Confirm that the batch biblio deletion tool still works correctly Created attachment 134648 [details] [review] Bug 30460: Make BatchDeleteBiblio update the index in one request No need to send 1 indexation request per item + 1 per biblio. Test plan: Confirm that the batch biblio deletion tool still works correctly Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> I'm torn by these patches.. they all improve reliability/reduce the possibility of overwhelming elasticsearch.. but they do so at the detriment of zebra by delaying indexing. As the general movement seems to be towards ES I'm going to PQA regardless :| Pushed to master for 22.05, thanks to everybody involved [U+1F984] Crap.. I didn't get Unit Tests added for any of these module changes.. |