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

WIP io parallel copy #834

Closed
wants to merge 3 commits into from
Closed

WIP io parallel copy #834

wants to merge 3 commits into from

Conversation

mmatczuk
Copy link
Contributor

@mmatczuk mmatczuk commented Jun 6, 2024

No description provided.

@mmatczuk mmatczuk requested a review from Choraden as a code owner June 6, 2024 14:58
@mmatczuk mmatczuk marked this pull request as draft June 6, 2024 14:58
@mmatczuk
Copy link
Contributor Author

I tested this patch with Forwarder and sc build, both locally and with dedicated Linux box 4 CPU Intel in local network.
I see no stable improvement that would justify the change.
In the test against gnet hello world, leading to this work, the new implementation (both local and over sc tunnel) does not change anything. In the test I'm able to get 20MiB/s throughput from a single core running gnet while forwarder and sc peak at 8,5MiB.

Testing against httpbin does not really work due to high variation in latencies stddev ranging to 70-100% both with wrk and vegeta.
In vegeta this can be flattened out by using HDR but one can note how stable gnet latencies are.

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.

1 participant