-
Notifications
You must be signed in to change notification settings - Fork 523
Conversation
d141dc9
to
657c51d
Compare
|
Pipelined 16
Non-pipelined
|
This is a lot in a single PR. Is it all really part of the same change or could you break this up into multiple PRs? |
I think its more of a journey that a PR :) I should be able to split it in to separate PRs though have some of them build of a merge of another? e.g. Only pooled blocks -> Expose pointer -> Removed fixed (in lots of places) |
85c4efa
to
ad32e4f
Compare
After last two
|
Not sure about last commit, but...
|
256 latency is crazy
|
|
|
efdf5ff
to
82e1218
Compare
238682a
to
02cafa1
Compare
|
Is there any particular reason that the ValueTask inclusion didn't make the combo? |
@halter73 not really :) There is a change to OutputSocket in this that is a little dodgy; sometimes good for throughput sometimes really bad. Have to rework it - but will do that as an individual PR |
@benaadams Since you've split this into smaller PRs, are we OK to close this now? |
Yeah; though is a bunch more it this that I haven't done PRs for; but I'll remember them. |
Best yet... (though will probably have to break it down into pieces?)
Azure G4 Peaking 2.7 GBps send 1.1GBps receive
344,334 rps Non pipelined w/ 3.44ms latency (32 threads and 1024 connections)
2,398,936 rps 16 pipelined w/ 5.66ms latency (32 threads and 1024 connections)
(3.9 - 4.2 Gbps send / 1.4 - 1.5 Gbps receive when pipelined 512)