mirror of
https://github.com/milvus-io/milvus.git
synced 2026-07-21 10:15:43 +00:00
aa548a5d0d
issue: #51253 pr: #51313 Combined cherry-pick of the two master fixes for `DescribeCollection` queried by collection id only (e.g. the HTTP management API on `:9091`): - #51254 — return the actual `db_name`/`db_id` from the proxy meta cache instead of echoing the (empty) request db name when `proxy.enableCachedServiceProvider` is on. - #51313 — two commits: 1. Backfill the top-level `collection_name` on both the cached provider path and the `describeCollectionTask` path; stop discarding the caller-provided collection id on the cached path; resolve identifiers on local copies instead of rewriting the request in place (access logs / metric labels read it after the handler returns). 2. Rework the proxy meta cache to be entity-keyed: fills land under the collection's actual database (the bogus `""` bucket for id-only lookups no longer exists, ending duplicate entries and same-name cross-database eviction), by-id lookups are served from a cluster-wide `collectionID → entry` index in O(1) (matching rootcoord, which ignores the db name for by-id describes), and empty-db name lookups normalize to `default` mirroring rootcoord's backward-compat normalization. All commits cherry-picked clean; no conflicts, no 3.0-specific adjustments. Verified on this branch: `TestMetaCache*`, `TestCachedProxyServiceProvider*`, `TestDescribeCollectionTask*`, alias and partition test sets all pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- ## Update: cherry-picked the id-primary meta cache rebuild from the master PR Two additional commits cherry-picked from #51313 (clean picks, same base state): the AlterAlias pre-alter target id forwarding (additive `old_collection_id` proto field) and the id-primary cache rebuild with O(1) invalidation. See #51313's description and `docs/design-docs/design_docs/20260716-proxy-metacache-id-inverted-index.md` for the full design. issue: #51533 --- ## Update: simplification series cherry-picked from #51313 (final form) Same commits as the master PR's id-primary simplification series (see #51313's description and the design doc for details, including the §6 WONT-FIX accepted gaps). Signed-off-by: xiaofanluan <xf@hjjaq.com> Co-authored-by: xiaofanluan <xf@hjjaq.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>