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

Already on GitHub? Sign in to your account

Updated tyrant-node with bug fixes. #3

Open
wants to merge 2 commits into
from

Conversation

Projects
None yet
1 participant

hoffin commented Feb 24, 2013

Updated tcp library name and calls. (For node.js 0.8.20)
Reduced minimum data size to 5 for responseMulti to handle empty results.
Fixed handling incomplete response from server by fixing checks for partial response.
(This was made evident using ab (Apache Batch) testing with 1000 concurrent requests.)

David Michael Updated to be compatible with Node 0.8.20.
  Updated tcp library name and calls.
  Reduced minimum data size for responseMulti to handle empty results.
  Fixed handling incomplete response from server by fixing checks for partial response.
3a9ba47

@hoffin hoffin commented on the diff Feb 24, 2013

tyrant.js
@@ -109,7 +109,7 @@ var queries = {
function decode_utf8(a) {
var string = "";
var i = 0;
- var c = c1 = c2 = 0;
+ var c = 0, c1 = 0, c2 = 0;
@hoffin

hoffin Feb 24, 2013

Made these variables local rather than global.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment