Bug 30277 - Make fetching ILL data from the API more efficient
Summary: Make fetching ILL data from the API more efficient
Status: RESOLVED DUPLICATE of bug 22440
Alias: None
Product: Koha
Classification: Unclassified
Component: ILL (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 32626
  Show dependency treegraph
 
Reported: 2022-03-11 13:30 UTC by Magnus Enger
Modified: 2023-04-16 14:18 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Magnus Enger 2022-03-11 13:30:36 UTC
Librarians are complaining that displaying the full table of ILL requests is taking a lot of time, even with moderate amounts of requests (on the order of 100-200). 

I think there are a couple of bottlenecks:

1. Embedding data

We are embedding 
- metadata
- patron
- capabilities
- library
- status_alias
- comments
- requested_partners

That's at least 6 additional queries times the number of requests in the table. Not sure what can be done here in general, though. 

2. Capabilities/status graph

For every request, the whole status graph is included with each request. When I look at the JSON data returned for one request in a sample database, it is about 540 lines of data. But about 400 of those make up the status graph, and since this system only has one active backend, those 400 lines are (as far as I can see) identical for all 200 requests. Could we send the status graph once per backend, and then have some reference/link from the request to the correct status graph? 

3. Fetching requests selectively

Today, data is fetched for all requests not hidden by ILLHiddenRequestStatuses before the table of requests is displayed. Bug 30200 proposes to add configurable tabs to the ILL requests table. Could it be possible to load the data for the tabs just-in-time, so that only the data for the first tab is loaded initially? Instead of loading everything just-in-case when the page is first shown. And then have the "All" tab to the right, so that "all" data is only loaded if the user asks for it.
Comment 1 Katrin Fischer 2023-04-16 12:29:51 UTC
Maybe this was fixed by some work from the hackfest?
Comment 2 Tomás Cohen Arazi 2023-04-16 14:18:59 UTC
This is already being worked on bug 22440. With great results. The UI details are looked after by Pedro

*** This bug has been marked as a duplicate of bug 22440 ***