-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Allow DbContextFactory to use a pool #21431
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make the leases structs to reduce allocations
You'd also need to collapse them into one type and get rid of the interface to avoid boxing |
src/EFCore/Extensions/EntityFrameworkServiceCollectionExtensions.cs
Outdated
Show resolved
Hide resolved
3a9070b
to
1dafcf0
Compare
@AndriySvyryd New version up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be easier to review as separate commits
Fixes #21247 Refactors the context pooling code to create a cleaner responsibility of services. Has the same restrictions as regular pooling since the context instances are reused.
1dafcf0
to
d7b9414
Compare
Fixes #21247
Refactors the context pooling code to create a cleaner responsibility of services.
Has the same restrictions as regular pooling since the context instances are reused.