Bug 32339

Summary: [Elasticsearch] Provide record when cataloguing biblios
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: CatalogingAssignee: Aleisha Amohia <aleisha>
Status: In Discussion --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: m.de.rooy, phil
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: Sponsored
Patch complexity: Small patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 32339: Provide record when cataloguing biblios

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.