Skip to content

fix(core): use future_scope to run parallel future#9637

Merged
hardfist merged 2 commits intoweb-infra-dev:mainfrom
quininer:fix-deadlock-with-scoped-token
Mar 13, 2025
Merged

fix(core): use future_scope to run parallel future#9637
hardfist merged 2 commits intoweb-infra-dev:mainfrom
quininer:fix-deadlock-with-scoped-token

Conversation

@hardfist
Copy link
Contributor

@hardfist hardfist commented Mar 12, 2025

Summary

future_scope is more safe than async_scope cause you need explicitly pass ref to it and reviewer just need to review whether the passed ref is still valid during future execution

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link

netlify bot commented Mar 12, 2025

Deploy Preview for rspack ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 00372f2
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67d166af7e41aa0008f8a551
😎 Deploy Preview https://deploy-preview-9637--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@quininer quininer force-pushed the fix-deadlock-with-scoped-token branch 2 times, most recently from b9e4cdf to e61ebc5 Compare March 12, 2025 07:33
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 12, 2025

CodSpeed Performance Report

Merging #9637 will not alter performance

Comparing quininer:fix-deadlock-with-scoped-token (00372f2) with main (a79e3c0)

Summary

✅ 8 untouched benchmarks

@quininer quininer force-pushed the fix-deadlock-with-scoped-token branch 3 times, most recently from dbf411f to e4bf660 Compare March 12, 2025 09:48
@quininer quininer force-pushed the fix-deadlock-with-scoped-token branch from e4bf660 to 00372f2 Compare March 12, 2025 10:49
@hardfist hardfist changed the title Fix deadlock with scoped token fix(core): use future_scope to run parallel future Mar 13, 2025
@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Mar 13, 2025
@hardfist hardfist requested review from LingyuCoder and h-a-n-a March 13, 2025 04:25
@hardfist hardfist merged commit 1c7554b into web-infra-dev:main Mar 13, 2025
37 checks passed
@quininer quininer deleted the fix-deadlock-with-scoped-token branch March 13, 2025 04:35
@hardfist hardfist mentioned this pull request Mar 14, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants