search

monospice / laravel-redis-sentinel-drivers

Redis Sentinel integration for Laravel and Lumen.
MIT License
101 stars 48 forks source link

Queue driver doesn't work in Laravel 5.2 and below #4

Closed cyrossignol closed 6 years ago

cyrossignol commented 7 years ago

As @jacovdbergh found in #3, the queue implementation in Laravel 5.2 and below does not support transactions over a set of Redis servers behind Sentinel. Here's the exception:

Cannot initialize a MULTI/EXEC transaction over aggregate connections.

The package will need to implement the LuaScripts added to Laravel 5.3. Besides this issue, the LuaScripts solve some other race conditions faced by the older Laravel Redis Queue implementation, so backporting these will improve the package in other ways as well.

cyrossignol commented 6 years ago

Closing due to lack of interest. The upcoming backport of features from the 2.x branch to 1.x fixes this issue by adding compatibility for transactions. Those seeking improved queue reliability using the Lua script implementation should upgrade to Laravel 5.3 or greater. The Laravel project itself stopped supporting 5.2 almost two years ago.