From 9d3501523c91bd0c9b4ce35d8e8a3dd1a0164fbb Mon Sep 17 00:00:00 2001
From: Krishnanand Thommandra <devtkrishna@gmail.com>
Date: Tue, 7 Nov 2017 01:01:29 -0800
Subject: [PATCH] restrict refresh ring to heartBeat (#1015)

* don't trace unless query is trace enabled

When 'nodetool settraceprobability' is used,
it's possible to get a response with trace information
even though client did not specifically mark the query
for tracing.

* added myself as an author

* restrict refresh ring to heartBeat

GetHosts uses queries on control connection. Any query error will
cause reconnection and refresh ring which calls GetHosts and thus
deadlocks.

Also it seems neater to have the ring refresh be done only by heartBeat.

Fixes #928
---
 control.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/control.go b/control.go
index 3aa9d61ce..acbec3139 100644
--- a/control.go
+++ b/control.go
@@ -381,7 +381,7 @@ func (c *controlConn) HandleError(conn *Conn, err error, closed bool) {
 		return
 	}
 
-	c.reconnect(true)
+	c.reconnect(false)
 }
 
 func (c *controlConn) getConn() *connHost {