[GitHub] [ignite] ptupitsyn commented on a change in pull request #9863: IGNITE-16530 Java thin: add heartbeats

[GitBox <gi...@apache.org>]

ptupitsyn commented on a change in pull request #9863:
URL: https://github.com/apache/ignite/pull/9863#discussion_r818724340



##########
File path: modules/core/src/main/java/org/apache/ignite/internal/client/thin/TcpClientChannel.java
##########
@@ -705,9 +724,65 @@ private ClientException handleIOError(String chInfo, @Nullable IOException ex) {
         return new ClientConnectionException("Ignite cluster is unavailable [" + chInfo + ']', ex);
     }
 
+    /**
+     * Initializes heartbeats.
+     *
+     * @param configuredInterval Configured heartbeat interval, in milliseconds.
+     * @return Heartbeat timer.
+     */
+    private Timer initHeartbeats(long configuredInterval) {
+        long heartbeatInterval = getHeartbeatInterval(configuredInterval);

Review comment:
       Unfortunately, there is no logging in Java thin client.




-- 
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