This repository has been archived by the owner on Dec 19, 2018. It is now read-only.
Pool TagHelperExecutionContexts
#674
Comments
|
Moving to Backlog. |
|
Doing this real quick. It's a 3% overall perf gain and relatively simple. |
NTaylorMullen
added a commit
that referenced
this issue
Feb 18, 2016
- Currently the `TagHelperScopeManager` creates a new `TagHelperExecutionContext` per `TagHelper` on a given page. With this change the max number of `TagHelperExecutionContext`s per page is the number of nested levels that exist. - Decided to add two tests validating that specific pieces of `TagHelperExecutionContext` are updated as expected. #674
NTaylorMullen
added a commit
that referenced
this issue
Feb 18, 2016
- Currently the `TagHelperScopeManager` creates a new `TagHelperExecutionContext` per `TagHelper` on a given page. With this change the max number of `TagHelperExecutionContext`s per page is the number of nested levels that exist. - Decided to add two tests validating that specific pieces of `TagHelperExecutionContext` are updated as expected. #674
NTaylorMullen
added a commit
that referenced
this issue
Feb 18, 2016
- Currently the `TagHelperScopeManager` creates a new `TagHelperExecutionContext` per `TagHelper` on a given page. With this change the max number of `TagHelperExecutionContext`s per page is the number of nested levels that exist. - Decided to add two tests validating that specific pieces of `TagHelperExecutionContext` are updated as expected. #674
NTaylorMullen
added a commit
that referenced
this issue
Feb 19, 2016
- Currently the `TagHelperScopeManager` creates a new `TagHelperExecutionContext` per `TagHelper` on a given page. With this change the max number of `TagHelperExecutionContext`s per page is the number of nested levels that exist. - Added two tests validating that specific pieces of `TagHelperExecutionContext` are updated as expected. #674
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Today our
TagHelperExecutionContexts aren't pooled and end up resulting in ~2.4% of the overall allocations of a page.The text was updated successfully, but these errors were encountered: