account->person adaption is slow
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Critical
|
Guilherme Salgado |
Bug Description
allenap:
person_
performance
jamesh:
The adapter should have been fairly cheap. [It was] changed it to do a
query every time. A quick improvement would be to change the
IAuthorizat
person objects (person may be None) that would get rid of the need for
account->person adaption in the security adapters entirely.
jamesh:
Since the LaunchpadPrincipal has already looked up the person, feeding
it directly to the IAuthorization adapter will probably still be faster.
I'd expect that the majority of the calls are coming from the security
adapter code
allenap:
checkAccoun
But making person_from_team faster would be a more general fix.
description: | updated |
Changed in launchpad-registry: | |
assignee: | nobody → salgado |
status: | Triaged → In Progress |
tags: | added: story-performance-improvement |
This will be a CP candidate.