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

added rate limiting options #54

Merged
merged 1 commit into from Jul 8, 2016
Merged

Conversation

GlenTiki
Copy link
Collaborator

@GlenTiki GlenTiki commented Jul 8, 2016

Boom! No change in perf.

Glens-MacBook-Pro-2:autocannon glen$ ./autocannon.js -c 1000 -d 30 http://localhost:3000
Running 30s test @ http://localhost:3000
1000 connections

Stat         Avg     Stdev     Max
Latency (ms) 51.7    18.15     631
Req/Sec      19183.6 1286.36   20975
Bytes/Sec    2.13 MB 144.66 kB 2.36 MB

575k requests in 30s, 63.88 MB read
6 errors (0 timeouts)

Fixes #27

Example usage:

Glens-MacBook-Pro-2:autocannon glen$ ./autocannon.js -R 10 -c 2 -a 100 http://localhost:3000
Running 100 requests test @ http://localhost:3000
2 connections

Stat         Avg     Stdev    Max
Latency (ms) 0.4     1.77     12
Req/Sec      10      2.24     15
Bytes/Sec    1.12 kB 254.04 B 1.73 kB

100 requests in 10s, 11.1 kB read

@mcollina
Copy link
Owner

mcollina commented Jul 8, 2016

LGTM @davidmarkclements?

@davidmarkclements
Copy link
Collaborator

this is really cool

my only thing is - I'd like to review the entire API - but that's another exercise, happy for merge

@GlenTiki GlenTiki merged commit 103fa77 into mcollina:master Jul 8, 2016
@GlenTiki
Copy link
Collaborator Author

GlenTiki commented Jul 8, 2016

Merging, lets open an issue for API review. 👍

@GlenTiki GlenTiki mentioned this pull request Jul 8, 2016
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.

None yet

3 participants