[GitHub] [ignite-3] isapego commented on a diff in pull request #1786: IGNITE-18809 Java client: Reconnect secondary endpoints in background

["isapego (via GitHub)" <gi...@apache.org>]

isapego commented on code in PR #1786:
URL: https://github.com/apache/ignite-3/pull/1786#discussion_r1134393343


##########
modules/client/src/main/java/org/apache/ignite/client/IgniteClientConfiguration.java:
##########
@@ -93,10 +99,22 @@ public interface IgniteClientConfiguration {
      */
     int reconnectThrottlingRetries();
 
+    /**
+     * Gets the background reconnect interval, in milliseconds. Set to {@code 0} to disable background reconnect.
+     *
+     * <p>Ignite balances requests across all healthy connections (when multiple endpoints are configured).
+     * Ignite also repairs connections on demand (when a request is made).
+     * However, "secondary" connections can be lost (due to network issues, or node restarts). This property controls how ofter Ignite
+     * client will check all configured endpoints and try to reconnect them in case of failure.
+     *
+     * @return Background reconnect interval, in milliseconds.
+     */

Review Comment:
   Should we mention default value here?



-- 
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: notifications-unsubscribe@ignite.apache.org

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