chrisdinn / brando

A Redis client written with Akka's IO package
Other
107 stars 24 forks source link

Batched requests can get sent out of order #69

Open apricequaltrics opened 5 years ago

apricequaltrics commented 5 years ago

Batched requests can get sent out of order when a given request fails to write to the tcp socket. The connection will get a Tcp.CommandFailed message and then retry the message. But at this point it is possible other requests have already been sent successfully.

Ray1988 commented 4 years ago

I wonder is there any update here?

chrisdinn commented 4 years ago

This bug should be straight-forward to fix but I haven't prioritized it. Would you like to try I fix it? I would happily share my thoughts on how and merge a PR/republish.

On Mon, Apr 20, 2020 at 1:35 PM Xinghua Li notifications@github.com wrote:

I wonder is there any update here?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/chrisdinn/brando/issues/69#issuecomment-616704806, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA2PE4P7KIVF4VMIXLUSETRNSBVZANCNFSM4GIWHRRQ .