joao-r-reis commented on code in PR #1875:
URL: 
https://github.com/apache/cassandra-gocql-driver/pull/1875#discussion_r2081645814


##########
policies.go:
##########
@@ -323,6 +323,7 @@ func (host *selectedHost) Info() *HostInfo {
 func (host *selectedHost) Mark(err error) {}
 
 // NextHost is an iteration function over picked hosts
+// Should return nil if SelectedHost is not Up to prevent endless query 
execution.

Review Comment:
   the driver checks if the host is not up, the function should "eventually" 
return nil regardless of what causes it.
   
   `// Should return nil eventually to prevent endless query execution.`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to