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

Memory leak in EntityMaterializerSource #31866

Closed
ajcvickers opened this issue Sep 26, 2023 · 0 comments
Closed

Memory leak in EntityMaterializerSource #31866

ajcvickers opened this issue Sep 26, 2023 · 0 comments
Labels
area-dbcontext area-model-building closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. customer-reported Servicing-approved type-bug
Milestone
6.0.24

Comments

@ajcvickers
Copy link
Contributor

Originally reported as #31539 (comment)
This was referenced Sep 26, 2023
Closed
Open
@ajcvickers ajcvickers self-assigned this Oct 3, 2023
@ajcvickers ajcvickers added this to the 6.0.x milestone Oct 3, 2023
ajcvickers added a commit that referenced this issue Oct 3, 2023
@ajcvickers
[release/6.0] Stop storing model references in the EntityMaterializer…
0d27fa7 
…Source

Fixes #31866

This is the second part of the memory leak fix from #31867

### Description

Storing references to the model resulted in a customer reported memory leak in multi-tenant application.

### Customer impact

Customer-reported large memory leak.

### How found

Customer reported.

### Regression

No.

### Testing

Existing tests cover that there is no regression with the change in cache location. Manual testing that the model is no longer referenced.

### Risk

Low. Also quirked.
@ajcvickers ajcvickers mentioned this issue Oct 3, 2023
Merged
@ajcvickers ajcvickers modified the milestones: 6.0.x, 6.0.24 Oct 11, 2023
@ajcvickers ajcvickers added Servicing-approved closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. and removed Servicing-consider labels Oct 11, 2023
@ajcvickers ajcvickers removed their assignment Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-dbcontext area-model-building closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. customer-reported Servicing-approved type-bug
Projects
None yet
Milestone
6.0.24
Development

No branches or pull requests
1 participant
@ajcvickers