From 4094812b39be7511d7db11d1418fe3892f436975 Mon Sep 17 00:00:00 2001
From: Rob Szumski <rob@robszumski.com>
Date: Tue, 9 Dec 2014 16:02:16 -0800
Subject: [PATCH] rafthttp: improve start/stop logging

---
 rafthttp/streamer.go | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/rafthttp/streamer.go b/rafthttp/streamer.go
index fa788a3dc..f0bc6833d 100644
--- a/rafthttp/streamer.go
+++ b/rafthttp/streamer.go
@@ -59,7 +59,7 @@ func startStreamServer(w WriteFlusher, to types.ID, term uint64, fs *stats.Follo
 		done: make(chan struct{}),
 	}
 	go s.handle(w)
-	log.Printf("rafthttp: stream server to %s at term %d starts", to, term)
+	log.Printf("rafthttp: starting server stream to %s at term %d", to, term)
 	return s
 }
 
@@ -73,8 +73,8 @@ func (s *streamServer) send(ents []raftpb.Entry) error {
 	case s.q <- ents:
 		return nil
 	default:
-		log.Printf("rafthttp: streamer reaches maximal serving to %s", s.to)
-		return fmt.Errorf("reach maximal serving")
+		log.Printf("rafthttp: maximum number of streams %s reached", s.to)
+		return fmt.Errorf("maximum number of streams reached")
 	}
 }
 
@@ -88,14 +88,14 @@ func (s *streamServer) stopNotify() <-chan struct{} { return s.done }
 func (s *streamServer) handle(w WriteFlusher) {
 	defer func() {
 		close(s.done)
-		log.Printf("rafthttp: stream server to %s at term %d is closed", s.to, s.term)
+		log.Printf("rafthttp: server streaming to %s at term %d has been stopped", s.to, s.term)
 	}()
 
 	ew := &entryWriter{w: w}
 	for ents := range s.q {
 		start := time.Now()
 		if err := ew.writeEntries(ents); err != nil {
-			log.Printf("rafthttp: write ents error: %v", err)
+			log.Printf("rafthttp: encountered error writing to server log stream: %v", err)
 			return
 		}
 		w.Flush()
@@ -149,7 +149,7 @@ func (s *streamClient) start(tr http.RoundTripper, u string, cid types.ID) error
 	}
 	s.closer = resp.Body
 	go s.handle(resp.Body)
-	log.Printf("rafthttp: stream client to %s at term %d starts", s.to, s.term)
+	log.Printf("rafthttp: starting client stream to %s at term %d", s.to, s.term)
 	return nil
 }
 
@@ -170,7 +170,7 @@ func (s *streamClient) isStopped() bool {
 func (s *streamClient) handle(r io.Reader) {
 	defer func() {
 		close(s.done)
-		log.Printf("rafthttp: stream client to %s at term %d is closed", s.to, s.term)
+		log.Printf("rafthttp: client streaming to %s at term %d is been stopped", s.to, s.term)
 	}()
 
 	er := &entryReader{r: r}
@@ -178,7 +178,7 @@ func (s *streamClient) handle(r io.Reader) {
 		ents, err := er.readEntries()
 		if err != nil {
 			if err != io.EOF {
-				log.Printf("rafthttp: read ents error: %v", err)
+				log.Printf("rafthttp: encountered error reading the client log stream: %v", err)
 			}
 			return
 		}