Closed (fixed)
Project:
JSON:API Extras
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Nov 2018 at 15:49 UTC
Updated:
19 Dec 2018 at 09:34 UTC
Jump to comment: Most recent, Most recent file


Comments
Comment #2
ndobromirov commentedComment #3
ndobromirov commentedHere is the promised patch...
Comment #4
ndobromirov commentedComment #5
ndobromirov commentedComment #6
e0ipsoCan you please profile the latest version of the -dev branch. It seems that there is a conflict with some recent changes.
#3016725: Do not use cache.static
Comment #7
e0ipsoComment #8
ndobromirov commentedWill do some time today.
Comment #9
ndobromirov commentedHere is the profile with the latest dev:

Here is the one with the micro-optimizations proposed in the other issue and the attached patch:

Comment #12
e0ipsoThanks! This was fixed.
Comment #13
ndobromirov commentedAwesome!
if I check the first benchmarks 474ms to the last committed ones 22ms - 21x improvement! :D