Skip to content

Modify garbage collection data sampler logic#1795

Open
umaannamalai wants to merge 3 commits into
mainfrom
modify-gc-data-sampler
Open

Modify garbage collection data sampler logic#1795
umaannamalai wants to merge 3 commits into
mainfrom
modify-gc-data-sampler

Conversation

@umaannamalai

Copy link
Copy Markdown
Contributor

This PR gates calls to gc.get_objects() based on whether the configured value of top_object_count_limit is greater than 0. This is done to avoid calls to gc.get_objects() if the user does not intend to track object types collected by the GC to reduce the chances of race conditions occurring by dumping the heap.

@umaannamalai
umaannamalai requested a review from a team as a code owner July 16, 2026 22:08
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 9 0 0 1.05s
✅ MARKDOWN markdownlint 7 0 0 0 1.53s
✅ PYTHON ruff 1064 0 0 0 1.13s
✅ PYTHON ruff-format 1064 0 0 0 0.42s
✅ YAML prettier 23 0 0 0 1.71s
✅ YAML v8r 23 0 0 6.05s
✅ YAML yamllint 23 0 0 0.82s

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@mergify mergify Bot added the tests-failing Tests failing in CI. label Jul 16, 2026
@codecov-commenter

codecov-commenter commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.05%. Comparing base (8735269) to head (0a4a5c6).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1795      +/-   ##
==========================================
- Coverage   82.07%   82.05%   -0.02%     
==========================================
  Files         216      216              
  Lines       26784    26783       -1     
  Branches     4239     4238       -1     
==========================================
- Hits        21982    21976       -6     
- Misses       3336     3340       +4     
- Partials     1466     1467       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@TimPansino TimPansino self-assigned this Jul 17, 2026
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Jul 17, 2026
@mergify

mergify Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants