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

malloc: refinements #18897

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

malloc: refinements #18897

wants to merge 1 commit into from

Conversation

reusee
Copy link
Contributor

@reusee reusee commented Sep 20, 2024

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #18896

What this PR does / why we need it:

refine malloc profiles and tests

@matrix-meow matrix-meow added the size/S Denotes a PR that changes [10,99] lines label Sep 20, 2024
@reusee reusee force-pushed the mallocprofile branch 2 times, most recently from 1236f44 to e06a53d Compare September 21, 2024 03:34
@reusee reusee changed the title malloc: scale full stack samples instead of showing stack ommitted samples malloc: refinements Sep 21, 2024
@jensenojs
Copy link
Contributor

2024-09-21T04:14:05.5217734Z # FAILED CASES:
2024-09-21T04:14:05.5218064Z --- FAIL: TestCheckedAllocator (22.59s)
2024-09-21T04:14:05.5218482Z --- FAIL: TestDefaultAllocator (59.65s)
2024-09-21T04:14:05.5218914Z --- FAIL: TestSyncPoolAllocator (127.18s)
2024-09-21T04:14:05.5219374Z --- FAIL: TestInuseTrackingAllocator (26.89s)
2024-09-21T04:14:05.5219843Z --- FAIL: TestLeaksTrackingAllocator (27.32s)
2024-09-21T04:14:05.5220496Z --- FAIL: TestMetricsAllocator (31.23s)
2024-09-21T04:14:05.5220914Z --- FAIL: TestProfileAllocator (28.93s)
2024-09-21T04:14:05.5221329Z --- FAIL: TestReadOnlyAllocator (27.11s)
2024-09-21T04:14:05.5221771Z --- FAIL: TestSizeBoundedAllocator (28.60s)

malloc: scale full stack samples instead of showing stack ommitted sample

malloc: use testing.AllocsPerRun
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement size/S Denotes a PR that changes [10,99] lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants