Josh,
Great detective work! The culprit is likely the bbag.js code as you have
mentioned. It sounds like you have made a major improvement. Really, the
whole thing needs to be moved into action_trigger and library settings
and eventually baked into Evergreen. Share your improvements as you make
them! I could imagine a path forward coding a custom JS carousel
(instead of using jcarousel) using some of the great Angular things.
Coupled with a server side component in the form of an additional
gateway-v1 listener to provide the JSON data as needed. Right now, the
bbag.js code downloads the entire bookbag and injects the ID's into a
hidden DOM element. Perhaps it's time to open that can of worms again?
Is there a LP bug for this?
-Blake-
Conducting Magic
MOBIUS
On 10/13/2018 10:05 AM, Josh Stompro wrote:
Hello, I’m moving this over to the dev list.
Using the batch method to flesh the titles from the bookbags does seem
to offer significant performance improvements for us. (Sending all bib
id’s in one array to open-ils.search.biblio.record.mods_slim.retrieve)
Measured in chrome 69.0.3497.100
Before: (hot cache) 19 second page load, 480 requests, 757kb transferred.
After: (hot cache) 6 second page load, 76 requests, 434kb transferred.
Difference: 68% reduction in page load time, 84% reduction in
requests, 42% reduction in transfer amount.
I haven’t dealt with getting the correct order yet,
open-ils.search.biblio.record.mods_slim.retrieve returns results in
bibid order.
Josh
*From:* Open-ils-general
<open-ils-general-boun...@list.georgialibraries.org> *On Behalf Of
*Josh Stompro
*Sent:* Friday, October 12, 2018 2:31 PM
*To:* Evergreen Discussion Group
<open-ils-gene...@list.georgialibraries.org>
*Subject:* [OPEN-ILS-GENERAL] Mobius Carousel - reducing number of
requests?
Hello, We have been using the carousel from Mobius for a while now.
And I just noticed how resource intensive it seems to be.
Here are the slides from the presentation that Bake created about it.
http://slides.mobiusconsortium.org/blake/bookcarousel/#/1
And the code
https://github.com/mcoia/mobius_evergreen/tree/master/bookbag_update
We have ~ 30 catalog machines that are set to reset every 5 minutes of
idle time (since people don’t always log themselves out.) When I
watch our catalog page load which currently has 3 sliders, I see 483
requests made from the browser. ~450 of those requests are from the
carousel, since it request each book bag, and then request info for
each bookbag item. And then request images for the visible items. So
our 30 catalog machines generate 162000 requests an hour.
I’m wondering if anyone has tried pre-generating the bookbag json data
on a schedule so the web page can do just one request for the titles?
It also seems like the only info that is used is the title, while
1-2kb of data gets returned about each item. So a lot of unused data
gets moved around. The script that generates the static data file
could do things like randomize the order of the items, so a static
bookbag list would show different titles whenever it is re-generated.
It looks like open-ils.search.biblio.record.mods_slim.retrieve can
accept an array of ID’s, so that seems like it would reduce the number
of requests, so that may be another way to reduce the number of requests.
Thanks
Josh
Lake Agassiz Regional Library - Moorhead MN larl.org
Josh Stompro | Office 218.233.3757 EXT-139
LARL IT Director | Cell 218.790.2110