redis-client-benchmarks icon indicating copy to clipboard operation
redis-client-benchmarks copied to clipboard

NodeJs benchmarks should be updated

Open akashdeep-singh opened this issue 7 years ago • 1 comments

I read your benchmarks, and the NodeJs ones caught my attention. I ran your code on my machine, and got the following results:

Elapsed time: 97258ms Commands per second: 10282

This was using node v6.10.1 LTS on a MacBook Air 13 inch from 2011, intel Core i5 / 2.30 GHz, Windows 10 Pro 10.016257, 10 GB ram with Redis 3.2.3 with the redis instance being accessed over a VPN connection. While this does not show a lot of improvement over your results, some of the pitfalls you pointed out (node not being able to handle 1000000 commands in the pipeline for instance) are eliminated, as is io.js . I'll try folloeing up on this, testing the benchmark on a platform closer to your tests' environment.

akashdeep-singh avatar Aug 19 '17 15:08 akashdeep-singh

 node ./nodejs_ioredis_performance.js 
Elapsed time: 589ms
Commands per second: 169779

node ./nodejs_node_redis_performance.js 
Elapsed time: 2458ms
Commands per second: 406835

node 10.6.0, Core i3-4160 CPU @ 3.60GHz, Arch Linux (kernel 4.17.5)

foi avatar Jul 11 '18 05:07 foi