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

process: improve rejections tracking performance #34862

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mmarchini
Copy link
Contributor

Fixes: #34851

Before landing
  • Explain why this works on commit message
  • Add comments to the code to explain what's going on
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

TODO: explain on commit message why this improves performance.

Fixes: nodejs#34851
@nodejs-github-bot nodejs-github-bot added the process Issues and PRs related to the process subsystem. label Aug 21, 2020
@mmarchini mmarchini added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 21, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 21, 2020
@nodejs-github-bot
Copy link
Collaborator

@mmarchini
Copy link
Contributor Author

Test failures look legit, but I can't reproduce locally

@matt4530
Copy link

matt4530 commented Sep 7, 2020

Just following up on this since it looks like the CI process got fixed (or at least it is no longer reporting). Can we re-run the CI to see if your patch passes?

@Uzlopak
Copy link
Contributor

Uzlopak commented May 6, 2024

@H4ad

We can close this PR now ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
process Issues and PRs related to the process subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Promise Rejections delay process exiting (and others) in processPromiseRejections
4 participants