[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2041: mod_pagespeed crashes from time to time

[GitBox <gi...@apache.org>]

jmarantz commented on issue #2041:
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2041#issuecomment-704206060


   Hi -- thanks for the stack-traces. I do not remember seeing this problem before, but would you be willing to try 2 things:
   
   1. upgrade to mod_pagespeed 1.14 to see if this resolves the problem
   2. switch from memcached to redis
   
   I can't promise either of these fixes the issue, but it would be good at least to confirm whether (a) this was fixed already in 1.14 and (b) whether this is specific to memcached or common to the redis/memcache implementations. If it is memcached-specific then you might be happy with Redis as an alternative anyway, and in either case we'll know more once you switch.
   
   In case that does not work, I've looked at the code, and this was a feature added to coalesce multiple asynchronous reads of the same key. I think in retrospect this should've been made an option in case something like this went awry. It might be possible to disable or workaround this code, which was added 4 years ago in this commit: https://github.com/apache/incubator-pagespeed-mod/commit/42098f8ed9398ef1ce37a337c2aee2bdd61ee946 .
   
   Ideally if anyone has time and wants a C++ challenge, trying to repro and fix this would be a great contribution!
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org