Skip to content

Add a few multiplexing microoptimizations#3918

Merged
vonzshik merged 1 commit intonpgsql:mainfrom
vonzshik:multiplexing-micro-optimizations
Aug 19, 2021
Merged

Add a few multiplexing microoptimizations#3918
vonzshik merged 1 commit intonpgsql:mainfrom
vonzshik:multiplexing-micro-optimizations

Conversation

@vonzshik
Copy link
Contributor

Reduced MultiplexingStats allocations.
Removed deprecated counter.
Moved two Settings.Multiplexing checks to the Debug.Assert.

@vonzshik vonzshik requested a review from roji as a code owner August 15, 2021 18:45
Copy link
Member

@roji roji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, these are good cleanups/optimizations!

Reduced MultiplexingStats allocations.
Removed deprecated counter.
Moved two Settings.Multiplexing checks to the Debug.Assert.
@vonzshik vonzshik force-pushed the multiplexing-micro-optimizations branch from b38c36f to 8ca6d0a Compare August 19, 2021 10:40
@vonzshik vonzshik enabled auto-merge (squash) August 19, 2021 10:41
@vonzshik vonzshik merged commit ca57e06 into npgsql:main Aug 19, 2021
@vonzshik vonzshik deleted the multiplexing-micro-optimizations branch August 19, 2021 11:00
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