Skip to content

Use native scheduler if defined in global scope#26554

Merged
sammy-SC merged 9 commits intofacebook:mainfrom
sammy-SC:sammy/include-native-scheduler
May 5, 2023
Merged

Use native scheduler if defined in global scope#26554
sammy-SC merged 9 commits intofacebook:mainfrom
sammy-SC:sammy/include-native-scheduler

Conversation

@sammy-SC
Copy link
Contributor

@sammy-SC sammy-SC commented Apr 5, 2023 •
edited
Loading

React Native in open source should use native scheduler if it is available.

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Apr 5, 2023
@react-sizebot
Copy link

react-sizebot commented Apr 5, 2023 •
edited
Loading

Comparing: 4a1cc2d...d0989c5

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 162.95 kB 162.95 kB = 51.54 kB 51.54 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 164.48 kB 164.48 kB = 52.03 kB 52.04 kB
facebook-www/ReactDOM-prod.classic.js = 553.05 kB 553.05 kB = 98.24 kB 98.24 kB
facebook-www/ReactDOM-prod.modern.js = 536.90 kB 536.90 kB = 95.56 kB 95.56 kB
oss-experimental/scheduler/cjs/scheduler.native.development.js +∞% 0.00 kB 17.64 kB +∞% 0.00 kB 4.87 kB
oss-experimental/scheduler/cjs/scheduler.native.production.min.js +∞% 0.00 kB 4.73 kB +∞% 0.00 kB 1.74 kB
oss-experimental/scheduler/index.native.js +∞% 0.00 kB 0.21 kB +∞% 0.00 kB 0.15 kB
oss-stable-semver/scheduler/cjs/scheduler.native.development.js +∞% 0.00 kB 17.64 kB +∞% 0.00 kB 4.87 kB
oss-stable-semver/scheduler/cjs/scheduler.native.production.min.js +∞% 0.00 kB 4.73 kB +∞% 0.00 kB 1.74 kB
oss-stable-semver/scheduler/index.native.js +∞% 0.00 kB 0.21 kB +∞% 0.00 kB 0.15 kB
oss-stable/scheduler/cjs/scheduler.native.development.js +∞% 0.00 kB 17.64 kB +∞% 0.00 kB 4.87 kB
oss-stable/scheduler/cjs/scheduler.native.production.min.js +∞% 0.00 kB 4.73 kB +∞% 0.00 kB 1.74 kB
oss-stable/scheduler/index.native.js +∞% 0.00 kB 0.21 kB +∞% 0.00 kB 0.15 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
oss-experimental/scheduler/cjs/scheduler.native.development.js +∞% 0.00 kB 17.64 kB +∞% 0.00 kB 4.87 kB
oss-experimental/scheduler/cjs/scheduler.native.production.min.js +∞% 0.00 kB 4.73 kB +∞% 0.00 kB 1.74 kB
oss-experimental/scheduler/index.native.js +∞% 0.00 kB 0.21 kB +∞% 0.00 kB 0.15 kB
oss-stable-semver/scheduler/cjs/scheduler.native.development.js +∞% 0.00 kB 17.64 kB +∞% 0.00 kB 4.87 kB
oss-stable-semver/scheduler/cjs/scheduler.native.production.min.js +∞% 0.00 kB 4.73 kB +∞% 0.00 kB 1.74 kB
oss-stable-semver/scheduler/index.native.js +∞% 0.00 kB 0.21 kB +∞% 0.00 kB 0.15 kB
oss-stable/scheduler/cjs/scheduler.native.development.js +∞% 0.00 kB 17.64 kB +∞% 0.00 kB 4.87 kB
oss-stable/scheduler/cjs/scheduler.native.production.min.js +∞% 0.00 kB 4.73 kB +∞% 0.00 kB 1.74 kB
oss-stable/scheduler/index.native.js +∞% 0.00 kB 0.21 kB +∞% 0.00 kB 0.15 kB

Generated by 🚫 dangerJS against d0989c5

gaearon
gaearon previously requested changes Apr 6, 2023
View reviewed changes
Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

Can we do this via package.json export conditions instead? Does RN have one?

@sammy-SC
Copy link
Contributor Author

sammy-SC commented Apr 17, 2023

I added React Native override for index.js. This override checks if nativeRuntimeScheduler exists.

Do I need to bump up NPM version of this package and update the version that React uses?

@sammy-SC sammy-SC requested a review from gaearon April 17, 2023 15:18
@sammy-SC sammy-SC force-pushed the sammy/include-native-scheduler branch from c0304f1 to 139e4eb Compare April 20, 2023 12:17
gaearon
gaearon reviewed Apr 20, 2023
View reviewed changes
@sammy-SC sammy-SC mentioned this pull request May 4, 2023
Open
@gaearon gaearon force-pushed the sammy/include-native-scheduler branch 3 times, most recently from 0738434 to 1fe57da Compare May 4, 2023 18:18
@gaearon gaearon force-pushed the sammy/include-native-scheduler branch from 1fe57da to 2460eb2 Compare May 4, 2023 18:22
@gaearon gaearon dismissed their stale review May 4, 2023 18:23

stale

gaearon
gaearon approved these changes May 4, 2023
View reviewed changes
Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

this looks good on my side but pls test if it works.

note it would require an npm release — but we could also instead vendor Scheduler bundles into RN like we do with React bundles, and then it wouldn't.

@sammy-SC sammy-SC merged commit b00e273 into facebook:main May 5, 2023
@sammy-SC sammy-SC deleted the sammy/include-native-scheduler branch May 5, 2023 13:01
github-actions bot pushed a commit that referenced this pull request May 5, 2023
@sammy-SC
Use native scheduler if defined in global scope (#26554)
c165512 
Co-authored-by: Dan Abramov <dan.abramov@gmail.com>

DiffTrain build for [b00e273](b00e273)
acdlite added a commit to acdlite/next.js that referenced this pull request May 7, 2023
@acdlite
Upgrade React to 18.3.0-canary-16d053d59-20230506
424cf23 
Fixes a bug where `useFormStatus` crashed during SSR.

Includes the following upstream changes:

- [16d053d59](https://github.com/facebook/react/commits/16d053d59) Add useFormStatus to server rendering stub ([vercel#26788](facebook/react#26788)) (Andrew Clark)
- [efb381bbf](https://github.com/facebook/react/commits/efb381bbf) [Release Script] Print a hint where to get the token ([vercel#26783](facebook/react#26783)) (dan)
- [b00e27342](https://github.com/facebook/react/commits/b00e27342) Use native scheduler if defined in global scope ([vercel#26554](facebook/react#26554)) (Samuel Susla)
- [783e7fcfa](https://github.com/facebook/react/commits/783e7fcfa) React DevTools 4.27.6 -> 4.27.7 ([vercel#26780](facebook/react#26780)) (Ruslan Lesiutin)
- [377c5175f](https://github.com/facebook/react/commits/377c5175f) DevTools: fix backend activation ([vercel#26779](facebook/react#26779)) (Ruslan Lesiutin)
@acdlite acdlite mentioned this pull request May 7, 2023
Merged
acdlite added a commit to acdlite/next.js that referenced this pull request May 7, 2023
@acdlite
Upgrade React to 18.3.0-canary-16d053d59-20230506
123efcf 
Fixes a bug where `useFormStatus` crashed during SSR.

Includes the following upstream changes:

- [16d053d59](https://github.com/facebook/react/commits/16d053d59) Add useFormStatus to server rendering stub ([vercel#26788](facebook/react#26788)) (Andrew Clark)
- [efb381bbf](https://github.com/facebook/react/commits/efb381bbf) [Release Script] Print a hint where to get the token ([vercel#26783](facebook/react#26783)) (dan)
- [b00e27342](https://github.com/facebook/react/commits/b00e27342) Use native scheduler if defined in global scope ([vercel#26554](facebook/react#26554)) (Samuel Susla)
- [783e7fcfa](https://github.com/facebook/react/commits/783e7fcfa) React DevTools 4.27.6 -> 4.27.7 ([vercel#26780](facebook/react#26780)) (Ruslan Lesiutin)
- [377c5175f](https://github.com/facebook/react/commits/377c5175f) DevTools: fix backend activation ([vercel#26779](facebook/react#26779)) (Ruslan Lesiutin)
acdlite added a commit to acdlite/next.js that referenced this pull request May 7, 2023
@acdlite
Upgrade React to 18.3.0-canary-16d053d59-20230506
8b0485e 
Fixes a bug where `useFormStatus` crashed during SSR.

Includes the following upstream changes:

- [16d053d59](https://github.com/facebook/react/commits/16d053d59) Add useFormStatus to server rendering stub ([vercel#26788](facebook/react#26788)) (Andrew Clark)
- [efb381bbf](https://github.com/facebook/react/commits/efb381bbf) [Release Script] Print a hint where to get the token ([vercel#26783](facebook/react#26783)) (dan)
- [b00e27342](https://github.com/facebook/react/commits/b00e27342) Use native scheduler if defined in global scope ([vercel#26554](facebook/react#26554)) (Samuel Susla)
- [783e7fcfa](https://github.com/facebook/react/commits/783e7fcfa) React DevTools 4.27.6 -> 4.27.7 ([vercel#26780](facebook/react#26780)) (Ruslan Lesiutin)
- [377c5175f](https://github.com/facebook/react/commits/377c5175f) DevTools: fix backend activation ([vercel#26779](facebook/react#26779)) (Ruslan Lesiutin)
kodiakhq bot pushed a commit to vercel/next.js that referenced this pull request May 7, 2023
@acdlite
Upgrade React to 18.3.0-canary-16d053d59-20230506 (#49402)
7e02f11 
Fixes a bug where `useFormStatus` crashed during SSR.

Includes the following upstream changes:

- [16d053d59](https://github.com/facebook/react/commits/16d053d59) Add useFormStatus to server rendering stub ([#26788](facebook/react#26788)) (Andrew Clark)
- [efb381bbf](https://github.com/facebook/react/commits/efb381bbf) [Release Script] Print a hint where to get the token ([#26783](facebook/react#26783)) (dan)
- [b00e27342](https://github.com/facebook/react/commits/b00e27342) Use native scheduler if defined in global scope ([#26554](facebook/react#26554)) (Samuel Susla)
- [783e7fcfa](https://github.com/facebook/react/commits/783e7fcfa) React DevTools 4.27.6 -> 4.27.7 ([#26780](facebook/react#26780)) (Ruslan Lesiutin)
- [377c5175f](https://github.com/facebook/react/commits/377c5175f) DevTools: fix backend activation ([#26779](facebook/react#26779)) (Ruslan Lesiutin)
@fatwong fatwong mentioned this pull request Feb 27, 2024
Closed
EdisonVan pushed a commit to EdisonVan/react that referenced this pull request Apr 15, 2024
@sammy-SC @gaearon @AndyPengc12
Use native scheduler if defined in global scope (facebook#26554)
30e6d58 
Co-authored-by: Dan Abramov <dan.abramov@gmail.com>
bigfootjon pushed a commit that referenced this pull request Apr 18, 2024
@sammy-SC
Use native scheduler if defined in global scope (#26554)
2d742d3 
Co-authored-by: Dan Abramov <dan.abramov@gmail.com>

DiffTrain build for commit b00e273.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gaearon gaearon gaearon approved these changes

@rickhanlonii rickhanlonii rickhanlonii approved these changes

@acdlite acdlite Awaiting requested review from acdlite

Assignees

No one assigned

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sammy-SC @react-sizebot @gaearon @rickhanlonii @facebook-github-bot