436679 - Use memcache to cache forum pages for non logged in users

Status: RESOLVED FIXED

(support.mozilla.org :: Knowledge Base Software, task)

Description

[Nelson Ko [:nkoth]]

I think we should implement this, even though the Netscalers are caching forum pages. Just as bug 395271 was needed, I think this should also be done for forums.

Any comment, anyone?

Does anyone have the query count for a page like: http://support.mozilla.com/tiki-view_forum_thread.php?locale=en-US&forumId=1&comments_parentId=7801 ?

Comment 1

[Nelson Ko [:nkoth]]

The cache should be invalidated whenever:

1) someone adds a post to the thread, as well as
2) when any post within the thread is changed. Otherwise, pagination and all that might be messed up.
3) any post within the thread is reported, or status (i.e. type in the db) changed.

Technically, this means when anyone is adding a post (a reply) to the parent id of 7801 in the example in this bug, and also whenever a post that has a parent id of 7801 is edited.

Comment 2

[Les Orchard [:lorchard]]

Attachment: Adding output caching to forum reads for anonymous users, with cache invalidation on edits and reporting

Attaching a patch that does output buffer caching on anonymous forum reads, with cross-locale cache invalidation on edits and report flagging.  Not sure where else thread modifications happen, so throwing this patch out there for now.

Comment 3

[Laura Thomson :laura]

Comment on attachment 333673 [details] [diff] [review]
Adding output caching to forum reads for anonymous users, with cache invalidation on edits and reporting

Works for me.

Comment 4

[Les Orchard [:lorchard]]

Checked in as r17686

Comment 5

[Laura Thomson :laura]

Les, can you please merge this to prod branch so I can tag it?

Comment 6

[Les Orchard [:lorchard]]

Merged to prod along with bug 448498 as r17708