microsoft / microsoft/gctoolkit
Function initialMark in CMSTenuredPoolParser show a bad use of getOccupancyBeforeAfterWithMemoryPoolSizeSummary
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 1.3k
- Forks
- 177
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 3
Description
The memory space does not change in initialMark. Moreover, this function is different of the initialMark in GenerationalHeapParser.java.
As far as I am concerned, MemoryPoolSummary tenured = trace.getOccupancyBeforeAfterWithMemoryPoolSizeSummary(4); should change to MemoryPoolSummary tenured = trace.getOccupancyWithMemoryPoolSizeSummary(4);.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with initialMark in CMSTenuredPoolParser and compare it with initialMark in GenerationalHeapParser.java. Inspect the trace call for pool 4 and verify that the implementation uses the occupancy summary without before/after data. Done means the corrected behavior is validated by the project's relevant existing tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- performance
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100