search

adrienmo / eredis_cluster

eredis_cluster is an erlang wrapper for eredis to support cluster mode of redis 3.0.0+
MIT License
79 stars 83 forks source link

Pipeline on several nodes #4

Closed paramahamsa closed 8 years ago

paramahamsa commented 8 years ago

Hi! Here are two changes: 1) In Redis 3.2.0 cluster slots was request was changed, so I add its handling with backward compatibility 2) Add qmn/1 - multi node request, that allows split pipe request by redis nodes (pools) and merge repsponses so it may look like pipe request.

adrienmo commented 8 years ago

Hi paramahamsa, thank you for your pull request ! I was not aware there was a change in the cluster slots command result for redis 3.2.0.

For the pipeline query on multiple server + reordering, I thought about it but I was too lazy to implement it. Thanks for your work!

I made some comment about the code you proposed. It would be great if you can have a look and tell me what you think about it.

paramahamsa commented 8 years ago

Hi, adrienmo! I've updated code in fork, please check it https://github.com/paramahamsa/eredis_cluster/commits/feature

adrienmo commented 8 years ago

Hi, paramahamsa,

Thanks for fixing your code ! I merged your pull request and released the version 0.5.7 of the library.