Opened 12 years ago
Closed 12 years ago
#1693 closed task (fixed)
Investigate performance of loading annotations from snapshots
Reported by: | Nicklas Nordborg | Owned by: | Nicklas Nordborg |
---|---|---|---|
Priority: | major | Milestone: | BASE 3.2 |
Component: | core | Version: | |
Keywords: | Cc: |
Description
There are indications (see http://baseplugins.thep.lu.se/ticket/398) that the fix for #1685 may have had a negative impact on performance. See also #1374 for information about the initial improvements with the snapshot system.
Change History (4)
comment:1 by , 12 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 12 years ago
comment:3 by , 12 years ago
(In [6065]) References #1693: Investigate performance of loading annotations from snapshots
Added a special case in the permission check that should avoid loading each annotation and item just for the permission check when we are only checking READ permission based on the annotation type. Test results for the above experiment are back to the original values.
comment:4 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
I have made some initial tests with an experiment that has 849 bioassays and 9 experimental factors. There is no clear difference for the initial caching (as expected). But after that, the loading times are about 3 times higher with the fix for #1685 in place as without the fix (3 seconds vs 1 second). So there is room for some improvement, even though the absolute loading times are not that bad.