Fixed: Redis-backed queues do not consume further messages once a queue is empty

fission / keda-connectors

Generic connectors for Keda which can be used as worker images as part of scaleTargetRef.

Apache License 2.0

23 stars 30 forks source link

Closed soharab-ic closed 1 month ago

soharab-ic commented 1 month ago

Redis LPop command does not wait for new messages.
Redis BLPop command is blocking version of LPop command and it blocks the connection when there are no elements to pop from the list.
Use BLPop for redis-http-connector instead of LPop.

sanketsudake commented 1 month ago

@soharab-ic can you confirm if we can reproduce https://github.com/fission/keda-connectors/issues/159 and have cross verified fix with the same ?

soharab-ic commented 1 month ago

@soharab-ic can you confirm if we can reproduce #159 and have cross verified fix with the same ?

Yes @sanketsudake, I reproduced the issue #159 and then verified the fix with the same.