FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: use setImmediate() in test-heapdump-shadowrealm.js by joyeecheung · Pull Request #49573 · nodejs/node · GitHub

/ node Public

test: use setImmediate() in test-heapdump-shadowrealm.js - #49573

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
joyeecheung:shadowrealm-test
Sep 11, 2023
Merged

test: use setImmediate() in test-heapdump-shadowrealm.js#49573
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
joyeecheung:shadowrealm-test

Conversation

Copy link
Copy Markdown
Member

With a tight loop the GC may not have enough time to kick in. Try setImmediate() instead.

Refs: #49572

With a tight loop the GC may not have enough time to kick in.
Try setImmediate() instead.
nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Sep 9, 2023
joyeecheung added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 9, 2023

Copy link
Copy Markdown
Member Author

I am not sure if stress test work on debian and fedora boxes, but here it is https://ci.nodejs.org/job/node-stress-single-test/424/

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 9, 2023

Copy link
Copy Markdown
Collaborator

joyeecheung added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 11, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 11, 2023
nodejs-github-bot merged commit dca4d84 into nodejs:main Sep 11, 2023

Copy link
Copy Markdown
Collaborator

Landed in dca4d84

ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
With a tight loop the GC may not have enough time to kick in.
Try setImmediate() instead.

PR-URL: #49573
Refs: #49572
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
This was referenced Sep 28, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL