roryqi commented on code in PR #13717:
URL: https://github.com/apache/gravitino/pull/13717#discussion_r4255259072


##########
clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClientConfiguration.java:
##########
@@ -66,8 +66,13 @@ public class GravitinoClientConfiguration {
   public static final String CLIENT_SOCKET_TIMEOUT_MS =
       GRAVITINO_CLIENT_CONFIG_PREFIX + "socketTimeoutMs";
 
+  /** Disables automatic transport retries, so ambiguous writes are surfaced 
to the caller. */
+  public static final String CLIENT_DISABLE_AUTOMATIC_RETRIES =

Review Comment:
   Why do we need this? If we want to disable the retry, we should use a config 
option `retry times`. `0` means not to retry.



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