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

[Javascript] HappyX binding #6887

Merged
merged 8 commits into from
Nov 5, 2023
Merged

[Javascript] HappyX binding #6887

merged 8 commits into from
Nov 5, 2023

Conversation

waghanza
Copy link
Collaborator

@waghanza waghanza commented Nov 4, 2023

No description provided.

@waghanza
Copy link
Collaborator Author

waghanza commented Nov 4, 2023

@Ethosa seems that the problem is on alpine using -slim variant is ok on my side.
maybe I could update results for monday with happyx :-)

@waghanza
Copy link
Collaborator Author

waghanza commented Nov 4, 2023

however, I use pm2 on node to use all cores, except for happyx, does the underlying server take care of using all cores ?

@Ethosa
Copy link
Contributor

Ethosa commented Nov 4, 2023

@waghanza yeap, happyx nodejs bindings compiled with httpx that should to support multi-core 🤔

@waghanza waghanza merged commit f9030a1 into master Nov 5, 2023
43 of 44 checks passed
@waghanza waghanza deleted the nodejs_happyx branch November 5, 2023 13:11
@waghanza
Copy link
Collaborator Author

waghanza commented Nov 5, 2023

When doing

wrk -H 'Connection: keep-alive' --connections 512 --threads 8 --duration 15 --timeout 1 --script /home/waghanza/workspace/benchmark/web/pipeline.lua http://172.17.0.2:3000/

I have

Running 15s test @ http://172.17.0.2:3000/
  8 threads and 512 connectionn
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency   427.03ms   29.22ms 850.81ms   78.06%
    Req/Sec   253.35    231.23     0.93k    71.04%
  27384 requests in 15.06s, 4.58MB read
  Socket errors: connect 0, read 0, write 0, timeout 66
  Non-2xx or 3xx responses: 13692\n
Requests/sec:   1818.65\n
Transfer/sec:    311.69KB\n

cc @Ethosa

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.

2 participants