Performance review of SandboxLink extension
Closed, DeclinedPublic
Actions

Assigned To

Authored By

	Legoktm
	Sep 7 2014, 12:18 AM

Description

Version: unspecified
Severity: normal

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:55 AM

• bzimport set Reference to bz70501.

• bzimport added a subscriber: Unknown Object (MLST).

There are only two things which could conceivably lead to performance
issues and I believe I've taken care of both of them.

Adding the personal sandbox link is (by default) disabled for anonymous users, whose page views are usually cached.
The extension uses the SkinPreloadExistence hook to avoid additional database queries about page existence status.

So who could review this? Assignee welcome.

I think Ori was going to sign off on it.

We don't require performance reviews pre-deployment anymore, so removing this as a blocker.

(Clearly no one is going to do this.)