double callback and connection timeouts about helenus

[devdazed]

this is happening on all versions of C* on, both the Thrift interface as well as CQL, and both versions of Node.

[calvinfo]

I think it was just some merge problem. All the pass now after returning when there are no available clients.

[calvinfo]

I just pushed to my master branch and am still seeing timeouts, though no longer getting the double callback problem because the callback is only executed once for the connect now.

All of the tests seem to complete quickly using the --report-timing flag:

NODE_PATH=lib/ node_modules/whiskey/bin/whiskey --real-time --scope-leaks --report-timing --tests "test/cql2.js test/cql3.js test/thriftHostPool.js test/thrift.js test/units.js"

However, I'm still seeing the same timeout. Is there something I'm missing about whiskey or what?

[devdazed]

the timeout could be due to a callback not getting fired

[devdazed]

since its connecting properly, I would say that maybe it is not calling back properly on close.

[calvinfo]

Ah - good call. I see the problem now. Give me a few minutes.

[calvinfo]

Never mind - I thought it was something caused by not emitting a close event: calvinfo@8c70911

But I'm still seeing the timeouts - still unclear why they are happening since all of the actual tests seem to pass with [OK]