+1 except for a minor issue: when I changed the name and it was still cached /p/test/blog/ would redirect to /p/test I added a commit to `ib/7824` with a fix and full explanation. If you're good with that, go ahead and merge & close. The caching looks good to me.
--- ** [tickets:#7824] Cache neighborhood record** **Status:** review **Milestone:** unreleased **Labels:** performance sf-current 42cc **Created:** Tue Feb 03, 2015 09:35 PM UTC by Dave Brondsema **Last Updated:** Fri Feb 06, 2015 05:52 PM UTC **Owner:** Igor Bondarenko Every single request hits `RootController.__init__` and does a neighborhood lookup. Neighborhoods don't change very often so this could be cached in-memory for a short time. I think the duration should be configurable and default to 0 (particularly so development is easy). Based on some performance monitoring I did, this may save up to 15% of time spent in a simple request (smaller relative % on more complex pages) --- Sent from forge-allura.apache.org because [email protected] is subscribed to https://forge-allura.apache.org/p/allura/tickets/ To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
