jnturton commented on code in PR #2655:
URL: https://github.com/apache/drill/pull/2655#discussion_r979835243


##########
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java:
##########
@@ -1094,6 +1095,35 @@ public static String bootDefaultFor(String name) {
       new OptionDescription("Enables recursive files listing when querying the 
`INFORMATION_SCHEMA.FILES` table or executing the SHOW FILES command. " +
         "Default is false. (Drill 1.15+)"));
 
+  public static final String STORAGE_PLUGIN_ACCESS_ATTEMPTS = 
"storage.plugin_access_attempts";
+  public static final PositiveLongValidator 
STORAGE_PLUGIN_ACCESS_ATTEMPTS_VALIDATOR = new PositiveLongValidator(
+    STORAGE_PLUGIN_ACCESS_ATTEMPTS,
+    10,
+    new OptionDescription(
+      "The maximum number of attempts that will be made to request metadata " +
+      "needed for query planning from a storage plugin."
+    )
+  );
+  public static final String STORAGE_PLUGIN_ATTEMPT_DELAY = 
"storage.plugin_access_attempt_delay";
+  public static final NonNegativeLongValidator 
STORAGE_PLUGIN_ATTEMPT_DELAY_VALIDATOR = new NonNegativeLongValidator(
+    STORAGE_PLUGIN_ATTEMPT_DELAY,
+    5*60*1000,

Review Comment:
   Done. I asked IDEA to reformat the code and it indented the subsequent lines 
in the string additions by one level. Let me know if you meant a bigger 
reformatting.



-- 
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: dev-unsubscr...@drill.apache.org

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

Reply via email to