isapego commented on a change in pull request #719:
URL: https://github.com/apache/ignite-3/pull/719#discussion_r826012533



##########
File path: 
modules/platforms/dotnet/Apache.Ignite/Internal/ClientFailoverSocket.cs
##########
@@ -252,5 +268,95 @@ private IEnumerable<IPAddress> GetIps(string host, bool 
suppressExceptions = fal
                 throw;
             }
         }
+
+        /// <summary>
+        /// Gets a value indicating whether a failed operation should be 
retried.
+        /// </summary>
+        /// <param name="exception">Exception that caused the operation to 
fail.</param>
+        /// <param name="op">Operation code.</param>
+        /// <param name="attempt">Current attempt.</param>
+        /// <returns>
+        /// <c>true</c> if the operation should be retried on another 
connection, <c>false</c> otherwise.
+        /// </returns>
+        private bool ShouldRetry(Exception exception, ClientOp op, int attempt)
+        {
+            var e = exception;
+
+            while (e != null && !(e is SocketException))
+            {
+                e = e.InnerException;
+            }
+
+            if (e == null)
+            {
+                // Only retry socket exceptions.

Review comment:
       Got it.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to