-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Labels
A-perfperformance issuesperformance issuesBroken WindowBugs / technical debt to be addressed immediatelyBugs / technical debt to be addressed immediatelyS-actionableSomeone could pick this issue up and work on it right nowSomeone could pick this issue up and work on it right now
Description
Seems that our priming caching infra is indeed at least confusing: #9704 (comment)
@jonas-schievink I wonder if you see some quick wins there? One reasonable option might be to only run cache priming on configuration change (that is, when we go non-quiescent -> quiescent). That's a bit tricky to code, because priming might get interrupted by edits, and we should re-run it in that case, but we still want to not re-run it when it completes.
Another option is to make salsa internally-parallel and just get rid of cache priming :o)
Originally posted by @matklad in #9564 (comment)
Metadata
Metadata
Assignees
Labels
A-perfperformance issuesperformance issuesBroken WindowBugs / technical debt to be addressed immediatelyBugs / technical debt to be addressed immediatelyS-actionableSomeone could pick this issue up and work on it right nowSomeone could pick this issue up and work on it right now