Problem/Motivation

It isn't possible to run the redis module with the Predis library because as soon there is a persistent lock needed it will throw an exception.

Proposed resolution

Add the missing implementation.

CommentFileSizeAuthor
#3 redis-predis-lock-test-fixes-2974737-3.patch5.9 KBBerdir
#2 add_predis_persitent_lock_backend-2974737-2.diff995 bytesErik Frèrejean
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Erik Frèrejean created an issue. See original summary.

Erik Frèrejean’s picture

Erik Frèrejean
Primary language Dutch
Location 🏴‍☠️
CreditAttribution: Erik Frèrejean at KennisMeester commented
Assigned: Erik Frèrejean » Unassigned
Status: Active » Needs review
FileSize
add_predis_persitent_lock_backend-2974737-2.diff995 bytes
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
FileSize
redis-predis-lock-test-fixes-2974737-3.patch5.9 KB

Thanks, I tried to investigate why tests on travis aren't failing, I was able to confirm the problem locally, but the lock tests just don't fail on travis, no matter what I do.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Fixed

  • Berdir committed b8c1e75 on 8.x-1.x 
    Issue #2974737 by Erik Frèrejean, Berdir: Predis, PersistentLock...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.