Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Turbopack] await shutdown before ending pool #75545

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

sokra
Copy link
Member

@sokra sokra commented Jan 31, 2025

What?

Make sure to gracefully shutdown the Turbopack process, by waiting for shutdown before killing the process.

This makes sure Persistent Caching will have time to store the cache, before it's killed. That should happen in background while doing SSG. (Maybe we need to change that in future for memory reasons, but it will work for now.)

Closes PACK-3887

@ijjk ijjk added created-by: Turbopack team PRs by the Turbopack team. type: next labels Jan 31, 2025
Copy link
Member Author

sokra commented Jan 31, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@sokra sokra changed the title await shutdown before ending pool [Turbopack] await shutdown before ending pool Jan 31, 2025
@sokra sokra requested a review from timneutkens January 31, 2025 23:19
@sokra sokra marked this pull request as ready for review January 31, 2025 23:19
@sokra sokra force-pushed the sokra/correctly-stop-turbopack branch from 9727c73 to 62498d5 Compare February 3, 2025 07:06
@ijjk
Copy link
Member

ijjk commented Feb 3, 2025

Tests Passed

@sokra sokra force-pushed the sokra/correctly-stop-turbopack branch from 62498d5 to 5fa05bf Compare February 3, 2025 11:21
@sokra sokra requested a review from mischnic February 4, 2025 17:59
@sokra sokra merged commit a281473 into canary Feb 5, 2025
130 of 131 checks passed
@sokra sokra deleted the sokra/correctly-stop-turbopack branch February 5, 2025 00:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants