You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pagespeed.apache.org by GitBox <gi...@apache.org> on 2018/05/08 15:41:06 UTC

[GitHub] jmarantz opened a new issue #1771: Redis DB index aliasing

jmarantz opened a new issue #1771: Redis DB index aliasing
URL: https://github.com/apache/incubator-pagespeed-mod/issues/1771
 
 
   Looking at the code, I believe that if two different VirtualHosts in the same config construct redis caches that differ only DB-index then the first one will win, and the second db-index will be ignored.
   
   n #1627 I think the db-index needs to be concatenated into the key used to lookup cache objects to be shared between multiple servers, in this block code in system_caches.cc:
   
   ```
       spec_signature =
           StrCat("r;", config->redis_server().ToString(), ";",
                  IntegerToString(config->redis_reconnection_delay_ms()), ";",
                  IntegerToString(config->redis_timeout_us()));
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services