Whenever the Mollom servers are unreachable and a new server list is requested (and stored to the db using variable_set), the cache is cleared.

Normal behaviour, but under heavy load this causes a loss of cached data and a serious performance penalty if the servers are unreachable for a certain amount of time.

Upon the next invocations of the module, this behaviour is repeated over and over and a lack of cached data can bring a site on its knees.

Maybe a counter has to be included so this is tried only X times or lookups should be disabled for X minutes?

Comments

dave reid’s picture

Version: 5.x-1.6 » 6.x-1.x-dev
Priority: Critical » Normal
Status: Active » Postponed (maintainer needs more info)

Not sure how exactly a variable_set() can bring a site to it's knees. And I'm not sure what you mean by the cache is cleared. Maybe introduce a sanity check in mollom() to not request the server list more frequently than 60 seconds. Otherwise, not sure what else we can do.

dave reid’s picture

Status: Postponed (maintainer needs more info) » Fixed

No response so marking as fixed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.