Skip to content

Conversation

@n8schloss
Copy link
Contributor

Normal priority is for things that we still want the user to see and idle pri is for things that are more speculative and may never resolve. We do not currently have a priority for things (like logging or sending data back to servers) where we want to ensure that it resolves, but where we want other things to be able to happen first. This adds such a priority.

I was debating a bit between calling it 'low' priority or 'logging' priority. I went with calling it 'low' because you can use this for way more than logging, but opening to changing it.

@sizebot
Copy link

sizebot commented Nov 8, 2018

Details of bundled changes.

Comparing: 051272f...da88002

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 +1.0% +0.7% 21.56 KB 21.77 KB 5.84 KB 5.88 KB NODE_DEV
scheduler.production.min.js 🔺+1.2% 🔺+1.1% 4.64 KB 4.7 KB 1.82 KB 1.84 KB NODE_PROD
Scheduler-dev.js +1.0% +0.6% 21.79 KB 22 KB 5.89 KB 5.92 KB FB_WWW_DEV
Scheduler-prod.js 🔺+0.9% 🔺+0.6% 13.06 KB 13.18 KB 2.86 KB 2.88 KB FB_WWW_PROD

Generated by 🚫 dangerJS

@sebmarkbage
Copy link
Collaborator

Seems reasonable to me. Thoughts, @acdlite?

@acdlite
Copy link
Collaborator

acdlite commented Nov 8, 2018

How about idle priority? :)

@acdlite
Copy link
Collaborator

acdlite commented Nov 8, 2018

Oh duh, we already that one. I thought it was still "offscreen" or something.

Let's merge and figure out the name later.

@acdlite acdlite merged commit 7c56013 into facebook:master Nov 8, 2018
jetoneza pushed a commit to jetoneza/react that referenced this pull request Jan 23, 2019
n8schloss added a commit to n8schloss/react that referenced this pull request Jan 31, 2019
@n8schloss n8schloss deleted the addLoggerPri branch February 11, 2019 21:00
@n8schloss n8schloss restored the addLoggerPri branch February 11, 2019 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants