cockroachdb / cockroachdb/cockroach

multitenant: handle descriptor removal in updateNodeAddress, updateStoreMap

Offen
#93,348 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
A-multitenancy C-enhancement T-kv
Vorherrschende Sprache
Go
Sterne
32.5k
Forks
4.1k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

These comments indicate that descriptors are added and updated but never removed.
https://github.com/cockroachdb/cockroach/blob/1144e9ae196547f3d533647d08cf18813da72a2b/pkg/ccl/kvccl/kvtenantccl/connector.go#L305
https://github.com/cockroachdb/cockroach/blob/1144e9ae196547f3d533647d08cf18813da72a2b/pkg/ccl/kvccl/kvtenantccl/connector.go#L325

Node count is used in this PR and it will be incorrect if nodes are removed.
https://github.com/cockroachdb/cockroach/pull/93325

Jira issue: CRDB-22278
Epic: CRDB-26091

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.