Skip to content

[10.0.0] Deduplicate repository mappings before sorting (#32066) - #32300

Merged
iancha1992 merged 2 commits into
bazelbuild:release-10.0.0from
bazel-io:cp32066-10.0.0-224004
Nov 2, 2026
Merged

iancha1992 merged 2 commits into
bazelbuild:release-10.0.0from
bazel-io:cp32066-10.0.0-224004

Conversation

@bazel-io

Copy link
Copy Markdown
Member

RepoMappingManifestAction deduplicates repository names in a HashMap before sorting the distinct names. It preserves the first mapping for a duplicate repository and keeps both expanded and compact output order; identical entry maps are still grouped for compact manifests.

In the original 32,768-package, 256-manifest fixture with 1,024 repositories, manifest generation improved from 1.829 to 1.483 s. The 32-repository control was unchanged. The complete patch matches the original and git diff --check passes upstream.

@fmeum

Closes #32066

COPYBARA_INTEGRATE_REVIEW=#32066 from dzbarsky:dzbarsky/upstream-bazel-151 a9a0cf8
PiperOrigin-RevId: 1008281270
Change-Id: I08f73a5516830412c2d81bf327db079c558ca4eb

Commit d4b6b22

RepoMappingManifestAction deduplicates repository names in a HashMap before sorting the distinct names. It preserves the first mapping for a duplicate repository and keeps both expanded and compact output order; identical entry maps are still grouped for compact manifests.

In the original 32,768-package, 256-manifest fixture with 1,024 repositories, manifest generation improved from 1.829 to 1.483 s. The 32-repository control was unchanged. The complete patch matches the original and git diff --check passes upstream.

@fmeum

Closes bazelbuild#32066

COPYBARA_INTEGRATE_REVIEW=bazelbuild#32066 from dzbarsky:dzbarsky/upstream-bazel-151 a9a0cf8
PiperOrigin-RevId: 1008281270
Change-Id: I08f73a5516830412c2d81bf327db079c558ca4eb
@bazel-io bazel-io added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Oct 30, 2026
@bazel-io
bazel-io requested a review from a team as a code owner October 30, 2026 22:40
@bazel-io bazel-io added the team-Configurability platforms, toolchains, cquery, select(), config transitions label Oct 30, 2026
@bazel-io
bazel-io requested review from fmeum and meisterT October 30, 2026 22:40
@bazel-io bazel-io added the awaiting-review PR is awaiting review from an assigned reviewer label Oct 30, 2026
@bazel-io
bazel-io requested review from dabanki and removed request for a team October 30, 2026 22:40
@iancha1992
iancha1992 enabled auto-merge October 30, 2026 22:41
@bazel-docs-pr-commenter

Copy link
Copy Markdown

❌ Bazel docs preview deployment failed (Mintlify result: timed_out).

Please check the GitHub Actions logs for details.

Updated for 3359345bafc1e386d50854bbcf4e7153561c717a

@iancha1992
iancha1992 added this pull request to the merge queue Nov 2, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 2, 2026
@iancha1992
iancha1992 enabled auto-merge November 2, 2026 20:11
@iancha1992
iancha1992 added this pull request to the merge queue Nov 2, 2026
Merged via the queue into bazelbuild:release-10.0.0 with commit e72597a Nov 2, 2026
36 checks passed
@github-actions github-actions Bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Configurability platforms, toolchains, cquery, select(), config transitions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants