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

Add a feature flag to enable expiration of retry lanes #27694

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

kassens
Copy link
Member

@kassens kassens commented Nov 13, 2023

An attempt to see if we can bring back expiration of retry lanes to avoid cases resolving Suspense can be starved by frequent updates.

In the past, this caused increase browser crashes, but a lot of time has passed since then. Just trying if we can re-enable this.

Old PR that reverted adding the timeout: #21300

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Nov 13, 2023
@kassens kassens requested a review from noahlemen November 13, 2023 21:48
An attempt to see if we can bring back expiration of retry lanes to avoid cases
resolving Suspense can be starved by frequent updates.
@kassens kassens force-pushed the enableRetryLaneExpiration branch from 57f829a to 560f7bf Compare November 13, 2023 21:49
Copy link
Contributor

@tyao1 tyao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we add the minimal repro as a regression test here or in a follow up? That will verify it fixes the issue and other feature flags don't affect it.

@kassens kassens merged commit 593ecee into facebook:main Nov 14, 2023
36 checks passed
@kassens kassens deleted the enableRetryLaneExpiration branch November 14, 2023 15:15
github-actions bot pushed a commit that referenced this pull request Nov 14, 2023
An attempt to see if we can bring back expiration of retry lanes to
avoid cases resolving Suspense can be starved by frequent updates.

In the past, this caused increase browser crashes, but a lot of time has
passed since then. Just trying if we can re-enable this.

Old PR that reverted adding the timeout:
#21300

DiffTrain build for [593ecee](593ecee)
kodiakhq bot pushed a commit to vercel/next.js that referenced this pull request Nov 15, 2023
EdisonVan pushed a commit to EdisonVan/react that referenced this pull request Apr 15, 2024
An attempt to see if we can bring back expiration of retry lanes to
avoid cases resolving Suspense can be starved by frequent updates.

In the past, this caused increase browser crashes, but a lot of time has
passed since then. Just trying if we can re-enable this.

Old PR that reverted adding the timeout:
facebook#21300
bigfootjon pushed a commit that referenced this pull request Apr 18, 2024
An attempt to see if we can bring back expiration of retry lanes to
avoid cases resolving Suspense can be starved by frequent updates.

In the past, this caused increase browser crashes, but a lot of time has
passed since then. Just trying if we can re-enable this.

Old PR that reverted adding the timeout:
#21300

DiffTrain build for commit 593ecee.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants