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
Comment #1
dave reidNot 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.
Comment #2
dave reidNo response so marking as fixed.