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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

WRK output wrong? #928

Closed
hjanuschka opened this issue Feb 18, 2019 · 2 comments
Closed

WRK output wrong? #928

hjanuschka opened this issue Feb 18, 2019 · 2 comments
Labels
question A question (converts to discussion)

Comments

@hjanuschka
Copy link

hey 馃 thx for this greate crate!
i am currently trying to figure out with http framework i should use, rocket seems to be solid.

when doing benchmarks, i have a little problem, wrk always show's all connections as "read" errors.

any idea what that could be?

Running 30s test @ http://127.0.0.1:8083
  12 threads and 13 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     2.34ms    2.82ms  87.67ms   93.04%
    Req/Sec   463.56     97.90     0.99k    66.53%
  166349 requests in 30.07s, 31.41MB read
  Socket errors: connect 0, read 166344, write 0, timeout 0
Requests/sec:   5531.90
Transfer/sec:      1.04MB

when doing, the same business logic, with nickel, iron or whatever,i get clean wrk output.
requests seem to work, if i do the test with ab results seem to be valid.

@jebrosen
Copy link
Collaborator

Try disabling keep-alive for the tests (ROCKET_KEEP_ALIVE=0 in the environment, or keep_alive = 0 in Rocket.toml). This looks similar to #580 and I'm inclined to think it's a bug in wrk's reporting or error handling; siege doesn't seem to have similar issues and neither do browsers.

It's also likely specific to hyper 0.10, which Rocket should stop using relatively soon. As such, I don't think we stand to gain much from keeping this issue open.

@jebrosen jebrosen added the question A question (converts to discussion) label Feb 18, 2019
@hjanuschka
Copy link
Author

@jebrosen THX! yes the keepalive env fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question A question (converts to discussion)
Projects
None yet
Development

No branches or pull requests

2 participants