Page MenuHomePhabricator

Emit cache setup issues from cache specs
ClosedPublic

Authored by epriestley on Apr 7 2015, 11:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 25, 9:37 AM
Unknown Object (File)
Fri, Dec 13, 10:07 PM
Unknown Object (File)
Fri, Dec 13, 12:42 AM
Unknown Object (File)
Sun, Dec 8, 3:03 AM
Unknown Object (File)
Tue, Dec 3, 8:08 PM
Unknown Object (File)
Nov 29 2024, 8:25 PM
Unknown Object (File)
Nov 26 2024, 12:01 PM
Unknown Object (File)
Nov 21 2024, 9:37 PM
Subscribers

Details

Summary

Ref T5501. Currently, we emit some bad warnings about, e.g., "apc.stat" on PHP 5.5+ systems with OPcache, where the warnings are not relevant.

Generate and raise warnings out of the CacheSpec pipeline so we only run relevant code.

Test Plan

Faked various warnings and saw them render correctly.

Diff Detail

Repository
rP Phabricator
Branch
cache5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5189
Build 5207: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Emit cache setup issues from cache specs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
src/applications/config/option/PhabricatorCoreConfigOptions.php
212

Oh, this also sets a default cache namespace.

If we don't do this, the cachekey table identifies all keys as belonging to some other namespace.

btrahan edited edge metadata.
btrahan added inline comments.
src/applications/cache/spec/PhabricatorCacheSpec.php
84

Will this eventually say "or a later version" or just kind of whatevs? Maybe we should maintain a single something with our recommend versions that we can refer folks to here?

Just musing about doc decay for this sort of thing...

This revision is now accepted and ready to land.Apr 8 2015, 5:17 PM

There will probably never be a later version, APC is essentially obsolete in modern PHP.

This revision was automatically updated to reflect the committed changes.