jenkins-bot has submitted this change and it was merged. Change subject: resourceloader: Set a TTL for minification cache entries ......................................................................
resourceloader: Set a TTL for minification cache entries Follows-up 458e7cabbbafd. HHVM's APC cache doesn't have any limitation or eviction logic. Thus entries without a TTL will linger indefinitely. At time of writing over 99% of APC entries were from "resourceloader:filter" because there are far more unpopular resources than popular ones. A fixed TTL is suboptimal as it will also cause popular resources (e.g. startup module, or jquery) to be re-minified every day. Beware this isn't one single minification. It's one minification, for every JS/CSS resource, on every app server, for many different lang/skin/config variations. Bug: T104769 Change-Id: Ib62c320da0eeede68aad3f2294006ec7dcf1df55 --- M includes/resourceloader/ResourceLoader.php 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Ori.livneh: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/resourceloader/ResourceLoader.php b/includes/resourceloader/ResourceLoader.php index 5d0ed3c..e6cb5eb 100644 --- a/includes/resourceloader/ResourceLoader.php +++ b/includes/resourceloader/ResourceLoader.php @@ -224,7 +224,8 @@ if ( $options['cacheReport'] ) { $result .= "\n/* cache key: $key */"; } - $cache->set( $key, $result ); + // Set a TTL since HHVM's APC doesn't have any limitation or eviction logic. + $cache->set( $key, $result, 24 * 3600 ); } catch ( Exception $e ) { MWExceptionHandler::logException( $e ); $this->logger->warning( 'Minification failed: {exception}', array( -- To view, visit https://gerrit.wikimedia.org/r/223343 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib62c320da0eeede68aad3f2294006ec7dcf1df55 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com> Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits