lafikl/consistent

remove host from ring

tinytub opened this issue · 1 comments

tinytub commented

consistent/consistent.go

Line 169 in 6e1d11f

func (c *Consistent) Remove(host string) bool {

Remove host from ring,but didn't remove from loadmap.
If use Hosts() to get all hosts, it return the list generate from the loadmap, not real hosts in the ring.
Whether or not delete the host from loadmap ?

lafikl commented

yes you're right.

a PR to rectify this issue would be welcome.