jorgebay / node-cassandra-cql

Node.js driver for Apache Cassandra
MIT License
144 stars 42 forks source link

result2 in callback function? #99

Open dest31337 opened 10 years ago

dest31337 commented 10 years ago

Trying to use node-cassandra-cql in async flow control methods which add for next function argument "callback" (and hides first argument "error"): async.waterfall([ function(callback) { client.executeAsPrepared(select, [param], callback ); }, function(result, callback) { ... }], ...); I got error because in Client.prototype._executeOnConnection callback called with 2 result args: callback(err, result1, result2); Why? Refer to documentation: "Callback should take two arguments err and result."

sarus commented 10 years ago

Didn't look into this too much but could this be similar to issue #85?

Perhaps result2 is actually the retryCount like in the execute function. The execute function also returns 3 parameters even though the documentation only states that it returns err and result.

dest31337 commented 10 years ago

It's undefined. I can't find any use of result2 in code.