Thomas Gries wrote:
The patchlet #1 solves the problem you are referring
to (at least in my
reference implementations and implementatation of other people).
...
No it doesn't, it reintroduces the bug, which you fixed (albeit
inefficiently) with a previous patch. That patch replaces the use of
UserTalkUpdate with a call to User::saveNewtalk(). The latter function
which was written by Evan for User.php 1.140, does not update the
user_touched field, which is the source of this bug. UserTalkUpdate
calls User::saveSettings(), which does update user_touched. Both
versions suffer from concurrency and replication lag problems due to
their use of $wgMemc->delete() instead of $wgMemc->set() to refresh the
cache.
-- Tim Starling