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 Nicklas Nordborg, 12 years ago

Owner: changed from everyone to Nicklas Nordborg
Status: newassigned

comment:2 by Nicklas Nordborg, 12 years ago

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.

comment:3 by Nicklas Nordborg, 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 Nicklas Nordborg, 12 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.