You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed earlier today that the cache looks to be to small for large internal apps, causing us to constantly reload summaries. This change fixes that, reducing build times significantly for large apps.
There is a larger issue here of the actual cache implementation though that should likely get resolved. We end up with several versions of each summary file in the cache until it fills up, so we will fill up the cache even on small apps today.
It is possible we should change how that works, instead of increasing this.
Fwiw, I see total ram consumption of almost exactly 1GB per worker after this change.
Change-Id: I0d8a06da72b86d44562819bc04f5ae5543610f0d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/107360
Reviewed-by: Konstantin Shcheglov <[email protected]>
Reviewed-by: David Morgan <[email protected]>
Commit-Queue: Jake Macdonald <[email protected]>
0 commit comments