-
Notifications
You must be signed in to change notification settings - Fork 30.3k
perf: optimize cell_data serialization format #88401
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
base: 01-09-migrate-to-typed-accessors
Are you sure you want to change the base?
perf: optimize cell_data serialization format #88401
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
| let (encode_fn, _) = value_type.bincode.ok_or_else(|| { | ||
| bincode::error::EncodeError::OtherString(format!( | ||
| "{} is not encodable", | ||
| value_type.global_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| let transient_entries: Vec<_> = self | ||
| .cell_type_max_index() | ||
| .map(|map| { | ||
| map.iter() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Stats from current PR🟢 1 improvement
📊 All Metrics📖 Metrics GlossaryDev Server Metrics:
Build Metrics:
Change Thresholds:
⚡ Dev Server
📦 Dev Server (Webpack) (Legacy)📦 Dev Server (Webpack)
⚡ Production Builds
📦 Production Builds (Webpack) (Legacy)📦 Production Builds (Webpack)
📦 Bundle SizesBundle Sizes⚡ TurbopackClient Main Bundles: **430 kB** → **430 kB** ✅ -135 B82 files with content-based hashes (individual files not comparable between builds) Server Middleware
Build DetailsBuild Manifests
📦 WebpackClient Main Bundles
Polyfills
Pages
Server Edge SSR
Middleware
Build DetailsBuild Manifests
Build Cache
🔄 Shared (bundler-independent)Runtimes
📝 Changed Files (10 files)Files with changes:
View diffsapp-page-exp..ntime.dev.jsfailed to diffapp-page-exp..time.prod.jsDiff too large to display app-page-tur..ntime.dev.jsfailed to diffapp-page-tur..time.prod.jsDiff too large to display app-page-tur..ntime.dev.jsfailed to diffapp-page-tur..time.prod.jsDiff too large to display app-page.runtime.dev.jsfailed to diffapp-page.runtime.prod.jsDiff too large to display pages-api-tu..time.prod.jsDiff too large to display pages-turbo...time.prod.jsDiff too large to display |
|
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
1 similar comment
|
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
Merging this PR will improve performance by 5.42%
Performance Changes
Comparing Footnotes
|
a31d0fb to
de4b361
Compare
8f4edd3 to
422aca4
Compare
de4b361 to
5cf311e
Compare
422aca4 to
f74d192
Compare
5cf311e to
cd8f1c3
Compare
f74d192 to
1bb53cb
Compare
cd8f1c3 to
4b82cc7
Compare
1bb53cb to
b50a4fc
Compare
4b82cc7 to
bbbc722
Compare
b50a4fc to
70993e5
Compare

No description provided.