[XL] Change in fuchsia/fuchsia[main]: [pkg-cache] Used packed map for MetaContents
0 views
Skip to first unread message
'Erick Tryzelaar (Gerrit)' via owners-override
Apr 17, 2026, 1:19:12 PM (3 days ago) Apr 17
to Owners Override
Attention needed from Owners Override
Erick Tryzelaar has uploaded the change for review![]( "Open in Gerrit")
Erick Tryzelaar would like Owners Override to review this change.
Commit message
[pkg-cache] Used packed map for MetaContents
This adds `PackedVec<T>`, which is similar to a `Vec<Box<[u8]>>` or
`Vec<String>`, and `PackedMap<K, V>`, which is similar to a
`BTreeMap<String, V>`. The keys are sorted and searched with a binary
search, but the optimized for fixed structures by inlining the keys into
a single `Vec` to reduce individual allocations and better memory
locality.
Furthermore, `MetaContents` is then updated to use `PackedMap` to reduce
memory usage serving the package directory.
Bug: 486283202
Change-Id: Ie2baa29416e75e779702cc20f2dba4dc6de2b7f6
Change diff
Change information
Files:
- M src/bringup/bin/bootpkg/BUILD.gn
- M src/bringup/bin/bootpkg/src/lib.rs
- M src/developer/ffx/plugins/package/archive/list/BUILD.gn
- M src/developer/ffx/plugins/package/archive/list/src/lib.rs
- M src/developer/ffx/plugins/package/archive/src/lib.rs
- M src/lib/BUILD.gn
- M src/lib/assembly/update_package/src/update_package.rs
- A src/lib/collections/packed/BUILD.gn
- A src/lib/collections/packed/OWNERS
- A src/lib/collections/packed/README.md
- A src/lib/collections/packed/src/lib.rs
- A src/lib/collections/packed/src/packed_item.rs
- A src/lib/collections/packed/src/packed_map.rs
- A src/lib/collections/packed/src/packed_map_builder.rs
- A src/lib/collections/packed/src/packed_vec.rs
- M src/sys/pkg/lib/fuchsia-pkg-testing/src/package.rs
- M src/sys/pkg/lib/fuchsia-pkg/BUILD.gn
- M src/sys/pkg/lib/fuchsia-pkg/src/build.rs
- M src/sys/pkg/lib/fuchsia-pkg/src/meta_contents.rs
- M src/sys/pkg/lib/fuchsia-pkg/src/package_manifest.rs
- M src/sys/pkg/lib/package-directory/BUILD.gn
- M src/sys/pkg/lib/package-directory/src/meta_as_dir.rs
- M src/sys/pkg/lib/package-directory/src/meta_subdir.rs
- M src/sys/pkg/lib/package-directory/src/non_meta_subdir.rs
- M src/sys/pkg/lib/package-directory/src/root_dir.rs
Change size: XL
Delta: 25 files changed, 2190 insertions(+), 185 deletions(-)
Related details
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/59428348156e8010db6d24c4ae603f1bd97a74fe-HTML%40fuchsia-review.googlesource.com.
'Erick Tryzelaar (Gerrit)' via owners-override
Apr 17, 2026, 1:19:16 PM (3 days ago) Apr 17
to Owners Override, Aaron Wood, Suraj Malhotra, David Koloski, Ben Keller, global-integrat...@fuchsia-infra.iam.gserviceaccount.com, fuchsia-int...@luci-project-accounts.iam.gserviceaccount.com
Attention needed from Owners Override
Erick Tryzelaar voted and added 1 comment![]( "Open in Gerrit")
Votes added by Erick Tryzelaar
| Fuchsia-Auto-Submit | +1 |
1 comment
Patchset-level comments
File-level comment, Patchset 34 (Latest):
Erick Tryzelaar . resolved
Adding owners-override to add a new top-level src/lib library.
Related details
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/59428348156e8010db6d24c4ae603f1bd97a74fe-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Adam Barth (Gerrit)' via owners-override
Apr 17, 2026, 2:04:08 PM (3 days ago) Apr 17
to Erick Tryzelaar, Owners Override, Aaron Wood, Suraj Malhotra, David Koloski, Ben Keller, global-integrat...@fuchsia-infra.iam.gserviceaccount.com, fuchsia-int...@luci-project-accounts.iam.gserviceaccount.com
Attention needed from Erick Tryzelaar and Owners Override
Adam Barth voted and added 1 comment![]( "Open in Gerrit")
Votes added by Adam Barth
| Owners-Override | +1 |
1 comment
Patchset-level comments
Adam Barth . resolved
Normally we prefer to add the OWNERS file in a separate commit.
Related details
Attention is currently required from:
- Erick Tryzelaar
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/2d8e6f19c2e5d2c93414da08c1758b063faeb649-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
Reply all
Reply to author
Forward
0 new messages