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

JTF collection uses lots memory inside VS #221

Closed
lifengl opened this issue Feb 26, 2018 · 0 comments
Closed

JTF collection uses lots memory inside VS #221

lifengl opened this issue Feb 26, 2018 · 0 comments
Labels
perf
Milestone
v15.7

Comments

@lifengl
Copy link
Member

lifengl commented Feb 26, 2018

JTF collection can use lots of memory in its two weakReference dictionaries. That could be related to the extensive usage of JTF to create dependencies between dataflow blocks, and others. The majority of those dictionaries are quite small, so should we consider a smaller initial capacity (or allow the code to create the collection to pick up a smaller initial size?)
@AArnott AArnott added the perf label Feb 27, 2018
AArnott added a commit that referenced this issue Feb 27, 2018
@AArnott
Allocate WeakKeyDictionary`2 with smaller initial capacities
5719bdb 
Fixes #221
@AArnott AArnott added this to the v15.7 milestone Feb 27, 2018
@AArnott AArnott mentioned this issue Feb 27, 2018
Merged
AArnott pushed a commit to AArnott/vs-threading that referenced this issue Oct 24, 2023
@dependabot
Bump powershell from 7.3.7 to 7.3.8 (microsoft#221)
646c179 
Bumps [powershell](https://github.com/PowerShell/PowerShell) from 7.3.7 to 7.3.8.
- [Release notes](https://github.com/PowerShell/PowerShell/releases)
- [Commits](PowerShell/PowerShell@v7.3.7...v7.3.8)

---
updated-dependencies:
- dependency-name: powershell
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@AArnott AArnott mentioned this issue Oct 24, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
perf
Projects
None yet
Milestone
v15.7
Development

No branches or pull requests
2 participants
@AArnott @lifengl