Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for multi/exec logic with detect_buffers enabled #733

Merged
merged 5 commits into from
Jul 12, 2015

Conversation

raydog
Copy link
Contributor

@raydog raydog commented Apr 1, 2015

Fixes #732 and #263.

Basically, fixes the bugs where:

  • Doing a .hgetall in a multi with detect_buffers while asking for strings mangles the data.
  • Doing a .hgetall in a multi with detect_buffers while asking for buffers throws an exception.
  • Asking for a mix of string/buffer responses will get you all string responses.

Thanks!

@raydog
Copy link
Contributor Author

raydog commented Apr 10, 2015

It's been around 10 days. Has anyone looked at this?

@raydog
Copy link
Contributor Author

raydog commented Apr 20, 2015

It's now been around 20 days. I don't mean to be a bother, but the README told me to be, especially at @DTrejo. :P

@raydog
Copy link
Contributor Author

raydog commented Jul 10, 2015

Due to the recent activity in the project.... bump? I'd love to get another pair of eyeballs on this. :)

@raydog
Copy link
Contributor Author

raydog commented Jul 11, 2015

Oh wait. I can bump this now with @NodeRedis/contributors :)

@mranney
Copy link
Contributor

mranney commented Jul 12, 2015

In fact, you hit the "Merge pull request" button, once you rebase it. :)

@raydog raydog force-pushed the multi-buffer-fix branch from 9b4288b to 19c5571 Compare July 12, 2015 05:49
@raydog
Copy link
Contributor Author

raydog commented Jul 12, 2015

lol, yeah. I was waiting to get the 👍 from someone other than myself, so I'll just take your comment as the 👍. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple bugs with multi/exec when detect_buffers is enabled.
2 participants