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

[v14.x backport] async_hooks: use resource stack for AsyncLocalStorage run #39974

Closed
wants to merge 1 commit into from

Conversation

Qard
Copy link
Member

@Qard Qard commented Sep 2, 2021

PR-URL: #39890
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Vladimir de Turckheim vlad2t@hotmail.com
Reviewed-By: Gerhard Stöbich deb2001-github@yahoo.de
Reviewed-By: Minwoo Jung nodecorelab@gmail.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com

PR-URL: nodejs#39890
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vladimir de Turckheim <vlad2t@hotmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@Qard Qard added async_hooks Issues and PRs related to the async hooks subsystem. v14.x labels Sep 2, 2021
@github-actions github-actions bot added the needs-ci PRs that need a full CI run. label Sep 2, 2021
@Qard Qard added request-ci Add this label to start a Jenkins CI on a PR. and removed needs-ci PRs that need a full CI run. labels Sep 2, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 2, 2021
@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member

targos commented Sep 2, 2021

The commit lands cleanly on my side and all tests pass. Did you change something to open this PR?

@Qard
Copy link
Member Author

Qard commented Sep 2, 2021

Ah, nope. It lands cleanly.

@targos
Copy link
Member

targos commented Sep 3, 2021

Okay, I pushed it in 5a42be9.

If you want to make sure a change lands on LTS, it should be sufficient to add the lts-watch-vx label and a comment on the original PR.

@targos targos closed this Sep 3, 2021
@Qard
Copy link
Member Author

Qard commented Sep 3, 2021

Ah, good to know, thanks! Sorry, I'm still learning how all the backporting stuff works. 😅

@Qard Qard deleted the backport-39890-to-14 branch September 3, 2021 14:55
@targos
Copy link
Member

targos commented Sep 4, 2021

No worries. I know everything is not clear about backporting.

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

Successfully merging this pull request may close these issues.

None yet

3 participants