https://bugzilla.wikimedia.org/show_bug.cgi?id=70585
Kunal Mehta (Legoktm) <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |PATCH_TO_REVIEW --- Comment #2 from Kunal Mehta (Legoktm) <[email protected]> --- (In reply to Aaron Schulz from comment #1) > * I don't like the unbounded cache in displayGlobalPage() I set it to 100, is that too high or too low? > * Seems like brokenLink() could check 'known' earlier to avoid DB queries > (the link methods are fairly hot) Done, and I further optimized it so it doesn't end up recursively calling itself. > * displayGlobalPage() loads user prefs to check 'globaluserpage' but we > don't have global preferences on Wrapped the check in a class_exists( 'GlobalPreferences' ) block > * getCentralTouched() has an unbounded cache and does not call > reuseConnection() Set the cache limit to 100, and I think I'm properly using reuseConnection, but not totally sure. > * memcached key in getRemoteParsedText() could be too long Wrapped the key in md5() > * Are we using $wgGlobalUserPageAPIUrl? If so, can it be configured to avoid > API queries? We have to use it for the action=parse request. We can use $wgConf for the remote url, so I changed that. > Maybe a timeout should be set for MWHttpRequest? The default $wgHTTPTimeout > might be a bit high. Also, what happens when it fails? I guess you get > warnings and null would get cached (the cache value check uses "!== false"). > Maybe the cache period could be reduced in that case and warnings avoided. The default timeout in core and on the cluster is 25, so I set it to 20. Should it be lower? Also adjusted the code to gracefully fail if the API request does. Change-Id: I7440151f29bf38b6c135d8508ac1a0cf24a5677e -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
