Skip to content

Commit 30b27ab

Browse files
committed
libnetwork/networkdb: drop id field from network
The map key for nDB.networks is the network ID. The struct field is not actually used anywhere in practice. Signed-off-by: Cory Snider <[email protected]>
1 parent 9a9cade commit 30b27ab

2 files changed

Lines changed: 3 additions & 7 deletions

File tree

libnetwork/networkdb/delegate.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,6 @@ func (nDB *NetworkDB) handleNetworkEvent(nEvent *NetworkEvent) bool {
136136

137137
// This remote network join is being seen the first time.
138138
nodeNetworks[nEvent.NetworkID] = &network{
139-
id: nEvent.NetworkID,
140139
ltime: nEvent.LTime,
141140
}
142141

@@ -452,10 +451,10 @@ func (d *delegate) LocalState(join bool) []byte {
452451
}
453452

454453
for name, nn := range d.nDB.networks {
455-
for _, n := range nn {
454+
for nid, n := range nn {
456455
pp.Networks = append(pp.Networks, &NetworkEntry{
457456
LTime: n.ltime,
458-
NetworkID: n.id,
457+
NetworkID: nid,
459458
NodeName: name,
460459
Leaving: n.leaving,
461460
})

libnetwork/networkdb/networkdb.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,6 @@ type node struct {
128128

129129
// network describes the node/network attachment.
130130
type network struct {
131-
// Network ID
132-
id string
133-
134131
// Lamport time for the latest state of the entry.
135132
ltime serf.LamportTime
136133

@@ -621,7 +618,7 @@ func (nDB *NetworkDB) JoinNetwork(nid string) error {
621618
if ok {
622619
entries = n.entriesNumber.Load()
623620
}
624-
nodeNetworks[nid] = &network{id: nid, ltime: ltime}
621+
nodeNetworks[nid] = &network{ltime: ltime}
625622
nodeNetworks[nid].entriesNumber.Store(entries)
626623
nodeNetworks[nid].tableBroadcasts = &memberlist.TransmitLimitedQueue{
627624
NumNodes: func() int {

0 commit comments

Comments
 (0)