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

Async implementation #2

Closed
wants to merge 20 commits into from
Closed

Async implementation #2

wants to merge 20 commits into from

Conversation

stebet
Copy link
Owner

@stebet stebet commented Mar 4, 2020

Async improvements

Stefán J. Sigurðarson and others added 18 commits February 24, 2020 13:59
Reducing memory usage when deserializing frames into commands.
ba14d84
Cleaning up CommandAssembler.
51e3683
Using recycled memory buffers when serializing frames.
4a653a6
@stebet
Removing unused code.
cdec7b4
@stebet
Updating the public API
82c7477
@stebet
Getting rid of an unnecessary NetworkBinaryWriter in the socket handler.
a5071a9
@stebet
Code cleanups.
5f6ec88
@stebet
Recycling command buffers to reduce allocations when receiving messages.
a7c443f
Merging with master.
3f05a3a
Removing unused variables and making readonly where possible.
4ca4acd
Removing last usages of NetworkBinaryReader and NetworkBinaryWriter a…
f6da18e 
…nd removing unused internal classes and tests. Adding NetworkByteOrder tests.
Merging with parent branch.
4d60eda
Updating public API.
8f931f3
Use ReadOnlyMemory where possible.
bad96c3
Removing the use of a BlockingCollection and replacing with Concurren…
62d6199 
…tQueue + SemaphoreSlim to get rid of a blocking wait for better concurrency.
Increasing buffer size for both send/receive for better throughput.
4e62101 
Removing use og SunchronizedList and replacing with just two longs to keep track of outstanding confirms.
Using async/await for Handle* methods.
0324424
Using static instances of BasicProperties when none are provided when…
b33d418 
… publishing.

Reducing socket buffer sizes.
Cleaning up AsyncConsumerDispatcher.
@stebet stebet mentioned this pull request Mar 4, 2020
Merged
11 tasks
@lukebakken lukebakken force-pushed the serializationAllocReductions branch 2 times, most recently from 5f33780 to b1c16b4 Compare March 4, 2020 20:13
lukebakken and others added 2 commits March 4, 2020 14:07
@stebet stebet closed this Mar 9, 2020
stebet pushed a commit that referenced this pull request Oct 19, 2023
@lukebakken
Misc Windows CI updates #2
5f155d1 
* Modify how RabbitMQ install dir is found in gha-run-tests.ps1
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.
2 participants
@stebet @lukebakken