mirror of
https://github.com/milvus-io/milvus.git
synced 2026-07-21 18:25:44 +00:00
39c9b039d2
Cherry-pick from master pr: https://github.com/milvus-io/milvus/pull/50221 pr: https://github.com/milvus-io/milvus/pull/50515 issue: #47420 Backports two related master PRs onto 2.6: - #50221 — standardize error handling on merr + Sys/Input classification (629 files) - #50515 — error-handling casebook + guide/convention docs (follow-up to #50221) ### 2.6 adaptation notes master has diverged significantly from 2.6 (whole-repo `pkg/v2`→`pkg/v3` and `milvus-proto/go-api/v2`→`v3` module renames, plus many feature PRs not on 2.6), so the replay required: - Normalizing all `pkg/v3` / `go-api/v3` imports back to `v2`. - Dropping edits to master-only files/features absent on 2.6 (snapshot, search aggregation, struct-array search, external-collection refresh, two-stage delegator, storagev2 packed FFI extras, function/chain, etc.). - Extending the raw-error→merr cleanup to the 2.6 files the newly added ruleguard linter flags (master had already cleaned these in earlier work). - Aligning a set of test error-code/message expectations with the new merr classification (mirroring #50221's own test changes that didn't apply cleanly across the divergence). ### Verification - build (go + cgo core) ✓ - make verifiers (gofmt + golangci-lint, all modules) ✓ - test-go: all changed packages + pkg module ✓ - e2e L0/L1/L2 (standalone): python_client + restful_client_v2 ✓ — remaining failures are environment/topology only (multi-querynode/datanode cluster tests, and the auth-enabled subset which passes once `authorizationEnabled=true`). --------- Signed-off-by: zhenshan.cao <zhenshan.cao@zilliz.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>