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.

Sign up for GitHub

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

Jump to bottom

Port latest wrk changes #79

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Port latest wrk changes #79

wants to merge 4 commits into from

Conversation

jonrichards
Copy link
Collaborator

  • Port wrk 4.1.0 changes
  • cherry-pick important updates from wrk master branch post-4.1.0 release

@jonrichards jonrichards mentioned this pull request Aug 27, 2019
Open
@jonrichards
Copy link
Collaborator Author

Sorry for the delays on this, but I have a Mac system now and I can reproduce this locally. I'll work on getting the build error resolved.

@jonrichards jonrichards changed the title WIP: Port latest wrk changes Port latest wrk changes Sep 21, 2019
@jonrichards
Copy link
Collaborator Author

This is ready for review @giltene @t-lo. Let me know how you'd like to proceed.

@lukeasrodgers
Copy link

FWIW - this compiles and runs for me on both Mojave and linux (ubuntu 16.04) 🎉 .

I get nan latency/rps for thread stats but that seems to be an existing issue with wrk2, not related to this port.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jonrichards @lukeasrodgers @wg