cockroachdb / cockroachdb/cockroach

multitenant: handle descriptor removal in updateNodeAddress, updateStoreMap

Aperta
#93,348 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
A-multitenancy C-enhancement T-kv
Lingua principale
Go
Stelle
32.5k
Fork
4.1k
Metriche di merge delle PR
Metriche PR in attesa

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.