[Bug 27343] New: Use L1 cache for get_all_subpermissions
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27343 Bug ID: 27343 Summary: Use L1 cache for get_all_subpermissions Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jonathan.druart@bugs.koha-community.org Reporter: jonathan.druart@bugs.koha-community.org QA Contact: testopia@bugs.koha-community.org Blocks: 27341 The permissions are fetched for every hit, we should cache them in L1. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27341 [Bug 27341] Performance 21.05 - Omnibus -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27343 --- Comment #1 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 114876 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114876&action=edit Bug 27343: Cache get_all_subpermissions return The permissions are fetched for every hit, we should cache them in L1. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27343 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #2 from David Cook <dcook@prosentient.com.au> --- What do you mean by every hit? It seems to me that it's basically called for every request, but we clear the L1 cache for every request anyway, so I'm not sure this patch would change anything? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27343 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |INVALID Status|ASSIGNED |RESOLVED --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Hum, looks like I was so frustrated to not find something significant in get_template_and_user that I invented problems. Yes, it's plain silly to use L1 in this situation. And L2 won't help much... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27343 --- Comment #4 from David Cook <dcook@prosentient.com.au> --- (In reply to Jonathan Druart from comment #3)
Hum, looks like I was so frustrated to not find something significant in get_template_and_user that I invented problems.
Yes, it's plain silly to use L1 in this situation. And L2 won't help much...
I do dislike get_template_and_user... heh. What about the block for "LoadSearchHistoryToTheFirstLoggedUser"? It looks like that could generate way more database calls than should be necessary, although I doubt that would show up in the benchmarking... Actually, going back to permissions, I have been thinking about whether we should calculate permissions for each request, or if we should calculate permissions at login time. I think that AWS calculates at least some permissions at login time, which would be more performant, although the annoying thing is that sometimes you have to logout and login again to gain newly assigned permissions. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org