Page MenuHomePhabricator

Remove PhabricatorStartup::getGlobal/setGlobal mechanism
ClosedPublic

Authored by epriestley on Jun 4 2015, 6:59 PM.
Tags
None
Referenced Files
F14051086: D13152.diff
Thu, Nov 14, 10:08 PM
F14046208: D13152.id31803.diff
Wed, Nov 13, 7:19 PM
F14038525: D13152.diff
Mon, Nov 11, 12:11 AM
F14023445: D13152.diff
Thu, Nov 7, 2:06 AM
F14008871: D13152.diff
Wed, Oct 30, 6:46 AM
F14001914: D13152.diff
Fri, Oct 25, 1:08 PM
F14000597: D13152.diff
Thu, Oct 24, 11:10 PM
F13989613: D13152.id.diff
Mon, Oct 21, 9:44 PM
Subscribers

Details

Summary

Ref T8424. Fixes T7114. This was envisioned as a per-request cache for reusing interpreters, but isn't a good fit for that in modern Phabricator.

In particular, it isn't loaded by the daemons, but they have equal need for per-request caching.

Since I finally need such a cache for Spaces, throw the old stuff away before I built a more modern cache.

Also resolves T7114 by dropping filtering on $_SERVER. I'm pretty sure this is the simplest fix, see D12977 for a bit more discussion.

Test Plan

Called didFatal() from somewhere in normal code and verified it was able to use the access log.

Diff Detail

Repository
rP Phabricator
Branch
spaces2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6544
Build 6566: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Remove PhabricatorStartup::getGlobal/setGlobal mechanism.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 4 2015, 9:15 PM
This revision was automatically updated to reflect the committed changes.