Merge ~cjwatson/launchpad:tolerate-pymemcache-connection-errors into launchpad:master
Proposed by
Colin Watson
Status: | Merged |
---|---|
Approved by: | Colin Watson |
Approved revision: | 2ca35a803d834602daba35a973355aa938998788 |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~cjwatson/launchpad:tolerate-pymemcache-connection-errors |
Merge into: | launchpad:master |
Diff against target: |
88 lines (+36/-3) 2 files modified
lib/lp/services/memcache/client.py (+3/-3) lib/lp/services/memcache/tests/test_memcache_client.py (+33/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jürgen Gmach | Approve | ||
Review via email: mp+412082@code.launchpad.net |
Commit message
Tolerate connection errors from pymemcache as well
Description of the change
We should (log and) ignore connection errors from pymemcache for the same reason as `MemcacheError`: it's an opportunistic cache, and a failure shouldn't cause the code trying to use it to fail.
To post a comment you must log in.
🤞