Skip to content

Commit 5e46b67

Browse files
committed
mmaprototype: remove RemoveNodeAndStores
It is unused and unclear what it would be used for. Easy to re-add should a need emerge.
1 parent c0ea6d4 commit 5e46b67

File tree

2 files changed

+0
-11
lines changed

2 files changed

+0
-11
lines changed

pkg/kv/kvserver/allocator/mmaprototype/allocator.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,6 @@ type Allocator interface {
3939
// about the nodes in the cluster is a side effect of this method.
4040
SetStore(store StoreAttributesAndLocality)
4141

42-
// RemoveNodeAndStores tells the allocator to remove the NodeID and all its
43-
// stores.
44-
RemoveNodeAndStores(nodeID roachpb.NodeID) error
45-
4642
// ProcessStoreLoadMsg provides frequent the state of every store and its
4743
// associated node in the cluster.
4844
ProcessStoreLoadMsg(ctx context.Context, msg *StoreLoadMsg)

pkg/kv/kvserver/allocator/mmaprototype/allocator_state.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -808,13 +808,6 @@ func (a *allocatorState) SetStore(store StoreAttributesAndLocality) {
808808
a.cs.setStore(store)
809809
}
810810

811-
// RemoveNodeAndStores implements the Allocator interface.
812-
func (a *allocatorState) RemoveNodeAndStores(nodeID roachpb.NodeID) error {
813-
a.mu.Lock()
814-
defer a.mu.Unlock()
815-
panic("unimplemented")
816-
}
817-
818811
// ProcessStoreLeaseholderMsg implements the Allocator interface.
819812
func (a *allocatorState) ProcessStoreLoadMsg(ctx context.Context, msg *StoreLoadMsg) {
820813
a.mu.Lock()

0 commit comments

Comments
 (0)