Bug 32339 - [Elasticsearch] Provide record when cataloguing biblios
Summary: [Elasticsearch] Provide record when cataloguing biblios
Status: In Discussion
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Aleisha Amohia
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-23 22:03 UTC by Aleisha Amohia
Modified: 2023-07-05 21:19 UTC (History)
2 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 32339: Provide record when cataloguing biblios (1.82 KB, patch)
2022-11-23 22:09 UTC, Aleisha Amohia
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Aleisha Amohia 2022-11-23 22:03:22 UTC
When reindexing with elasticsearch, the record is passed for authorities but not for biblios. This means that authorities are indexed as they are, while biblios are pushed to a background job. These processes should be consistent.
Comment 1 Aleisha Amohia 2022-11-23 22:09:01 UTC
Created attachment 144214 [details] [review]
Bug 32339: Provide record when cataloguing biblios

To test:

1. Ensure you have Elasticsearch enabled and configured
2. Add a new record and confirm you can search for it straight after
saving, before and after applying the patch

Sponsored-by: Auckland University of Technology
Comment 2 Katrin Fischer 2022-11-24 09:16:07 UTC
I am not sure this is the right fix, as we are moving to background jobs everywhere, maybe the indexing for the authorities should be changed instead?

Is there a visible difference here? My impression was that usually the single jobs get handled immediately.
Comment 3 Aleisha Amohia 2022-11-24 21:55:41 UTC
Also agree it might not be the best solution, I guess it might just be a workaround. The real problem for us is background jobs, they keep getting stuck at New and I found it was the queuing of the background job that meant records weren't getting indexed. We can cancel this if the ideal is that $record is never passed like it is for authorities.
Comment 4 Phil Ringnalda 2023-07-05 21:19:48 UTC
Sounds more like "in discussion" then.