From 08d4511e6c5da0ba399259579601cd1ff0c8237e Mon Sep 17 00:00:00 2001 From: Matt Jaffee Date: Fri, 5 Mar 2021 12:51:39 -0600 Subject: [PATCH] fix issue where restarting a cluster logged "invalid pilosa.Message" we were sending pilosa.Message objects from a spool, but actually passing a pointer to them rather than the Message itself. I'm concerned this wasn't caught in any test, and also curious if that needed to be a pointer for some reason or if it's a typo. Definitely need to write a test still. --- server.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server.go b/server.go index bd08550f0..fcd70ef13 100644 --- a/server.go +++ b/server.go @@ -679,7 +679,7 @@ func (s *Server) Open() error { s.logger.Printf("start initial cluster state sync") for i := range toSend { for { - err := s.holder.broadcaster.SendSync(&toSend[i]) + err := s.holder.broadcaster.SendSync(toSend[i]) if err != nil { s.logger.Printf("failed to broadcast startup cluster message (trying again in a bit): %v", err) timer.Reset(time.Second)