reduce duplicate rdy updation requests

nsqio · Feb 21, 2020 · 5254c93 · 5254c93
1 parent d9600b3
commit 5254c93
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/consumer.go b/consumer.go
@@ -981,7 +981,13 @@ func (r *Consumer) sendRDY(c *Conn, count int64) error {
 	}
 
 	atomic.AddInt64(&r.totalRdyCount, count-c.RDY())
+
+	lastRDY := c.LastRDY()
 	c.SetRDY(count)
+	if count == lastRDY {
+		return nil
+	}
+
 	err := c.WriteCommand(Ready(int(count)))
 	if err != nil {
 		r.log(LogLevelError, "(%s) error sending RDY %d - %s", c.String(), count, err)

diff --git a/mock_test.go b/mock_test.go
@@ -343,7 +343,6 @@ func TestConsumerRequeueNoBackoff(t *testing.T) {
 		"RDY 0",
 		fmt.Sprintf("REQ %s 0", msgIDRequeueNoBackoff),
 		"RDY 1",
-		"RDY 1",
 		fmt.Sprintf("FIN %s", msgIDGood),
 	}
 	if len(n.got) != len(expected) {