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

Move DbContext Pooling from Miscellaneous TOC to DbContext config #3656

Merged
merged 1 commit into from
Jan 4, 2022

Conversation

roji
Copy link
Member

@roji roji commented Jan 4, 2022

DbContext Pooling used to be a page under Miscellaneous, but was moved to the Advanced Performance Topics page under Performance. A redirection is already in place, but the old TOC entry was left in place.

@roji roji requested a review from a team January 4, 2022 09:36
@ajcvickers
Copy link
Member

Maybe we should leave it in both places? For discoverability.

@roji
Copy link
Member Author

roji commented Jan 4, 2022

I guess... I like to leave Miscellaneous as small as possible, and DbContext pooling really is a perf feature. If anything, it should probably be mentioned somewhere under "DbContext Lifetime, Configuration, and Initialization"...

@ajcvickers
Copy link
Member

ajcvickers commented Jan 4, 2022

I like putting it under "DbContext configuration and initialization"

@roji
Copy link
Member Author

roji commented Jan 4, 2022

I can do that. What do you think, an additional short section in the existing overview page? Because there's only that one page in the section, it would be weird to have a second DbContext Pooling TOC item...

@ajcvickers
Copy link
Member

There's going to be many more pages. As soon as I write them. :-) I think just move the second TOC item there.

@roji roji force-pushed the DbContextPoolingTOC branch 2 times, most recently from c0628b8 to f5a8dea Compare January 4, 2022 17:08
@roji
Copy link
Member Author

roji commented Jan 4, 2022

@ajcvickers done

@roji roji merged commit 85d7c82 into main Jan 4, 2022
@roji roji deleted the DbContextPoolingTOC branch January 4, 2022 18:07
@roji roji changed the title Remove DbContext Pooling from Miscellaneous TOC Move DbContext Pooling from Miscellaneous TOC to DbContext config Jan 4, 2022
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