ellemouton / lnd / 13152410111

Pull #178

github

graph: refactor Builder network message handling

The point of the `graph.Builder`'s `networkHandler` goroutine is to
ensures that certain requests are handled in a synchronous fashion.
However, any requests received on the `networkUpdates` channel, are
currently immediately handled in a goroutine which calls
`handleNetworkUpdate` which calls `processUpdate` before doing topology
notifications. In other words, there is no reason for these
`networkUpdates` to be handled in the `networkHandler` since they are
always handled asynchronously anyways. This design is most likely due to
the fact that originally the gossiper and graph builder code lived in
the same system and so the pattern was copied across.

173 of 268 new or added lines in 4 files covered. (64.55%)

100608 of 204163 relevant lines covered (49.28%)

1.54 hits per line