Skip to content

[scheduler] Rename priority levels #13842

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

Merged
merged 1 commit into from
Oct 12, 2018

Conversation

Image for: Conversation
Copy link
Collaborator

acdlite commented Oct 12, 2018

  • "Interactive" -> "User-blocking"
  • "Whenever" -> "Idle"

These are the terms used by @spanicker in their main-thread scheduling proposal: https://github.com/spanicker/main-thread-scheduling#api-sketch

That proposal also uses "microtask" instead of "immediate" and "default" instead of "normal." Not sure about "microtask" because I don't think most people know what that is. And our implementation isn't a proper microtask, though you could use it to implement microtasks if you made sure to wrap every entry point. I don't really have a preference between "default" and "normal."

These aren't necessarily the final names. Still prefixed by unstable_.

- "Interactive" -> "user-blocking"
- "Whenever" -> "Idle"

These are the terms used by @spanicker in their main-thread scheduling
proposal: https://github.com/spanicker/main-thread-scheduling#api-sketch

That proposal also uses "microtask" instead of "immediate" and "default"
instead of "normal." Not sure about "microtask" because I don't think
most people know what that is. And our implementation isn't a proper
microtask, though you could use it to implement microtasks if you made
sure to wrap every entry point. I don't really have a preference between
"default" and "normal."

These aren't necessarily the final names. Still prefixed by `unstable_`.
Copy link

sizebot commented Oct 12, 2018

Details of bundled changes.

Comparing: 3b7ee26...6073e96

scheduler

Image for: scheduler
File Filesize Diff Gzip Diff Prev Size Current Size Prev Gzip Current Gzip ENV
scheduler.development.js n/a n/a 0 B 19.17 KB 0 B 5.74 KB UMD_DEV
scheduler.production.min.js n/a n/a 0 B 3.16 KB 0 B 1.53 KB UMD_PROD
scheduler.development.js -0.2% 0.0% 22.32 KB 22.27 KB 6.01 KB 6.01 KB NODE_DEV
scheduler.production.min.js -0.1% 0.0% 4.81 KB 4.81 KB 1.88 KB 1.88 KB NODE_PROD
Scheduler-dev.js -0.2% 0.0% 22.58 KB 22.53 KB 6.04 KB 6.04 KB FB_WWW_DEV
Scheduler-prod.js -0.0% 0.0% 13.64 KB 13.64 KB 2.92 KB 2.92 KB FB_WWW_PROD

Generated by 🚫 dangerJS

Copy link
Collaborator

gaearon left a comment

Choose a reason for hiding this comment

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

i like

acdlite merged commit c9be16f into facebook:master Oct 12, 2018
acdlite deleted the rename-scheduler-priority-levels branch October 12, 2018 21:42
Copy link
Contributor

bvaughn commented Oct 12, 2018

me too

linjiajian999 pushed a commit to linjiajian999/react that referenced this pull request Oct 22, 2018
- "Interactive" -> "user-blocking"
- "Whenever" -> "Idle"

These are the terms used by @spanicker in their main-thread scheduling
proposal: https://github.com/spanicker/main-thread-scheduling#api-sketch

That proposal also uses "microtask" instead of "immediate" and "default"
instead of "normal." Not sure about "microtask" because I don't think
most people know what that is. And our implementation isn't a proper
microtask, though you could use it to implement microtasks if you made
sure to wrap every entry point. I don't really have a preference between
"default" and "normal."

These aren't necessarily the final names. Still prefixed by `unstable_`.
gaearon mentioned this pull request Oct 23, 2018
jetoneza pushed a commit to jetoneza/react that referenced this pull request Jan 23, 2019
- "Interactive" -> "user-blocking"
- "Whenever" -> "Idle"

These are the terms used by @spanicker in their main-thread scheduling
proposal: https://github.com/spanicker/main-thread-scheduling#api-sketch

That proposal also uses "microtask" instead of "immediate" and "default"
instead of "normal." Not sure about "microtask" because I don't think
most people know what that is. And our implementation isn't a proper
microtask, though you could use it to implement microtasks if you made
sure to wrap every entry point. I don't really have a preference between
"default" and "normal."

These aren't necessarily the final names. Still prefixed by `unstable_`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants