jrief / django-websocket-redis

Websockets for Django applications using Redis as message queue
http://django-websocket-redis.awesto.com/
MIT License
895 stars 221 forks source link

Add pipeline_publish_message method to redis_store #278

Closed mlindemu closed 1 year ago

mlindemu commented 5 years ago
mkaluza commented 5 years ago

Hi! I've stumbled upon a similar perf problem and came up with this: https://github.com/mkaluza/django-websocket-redis/tree/publisher_connection_override

Wouldn't this be simpler and more flexible? It allows you to publish many different messages with only one roundtrip instead of just one - you just pass the pipeline as connection.

And to mimic the functionality of your PR, I'd add a pipeline bool parameter to the constructor instead of adding a new method - that would make it easier to migrate existing applications to pipelines