Github user ilooner commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1119#discussion_r167346417
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
 ---
    @@ -212,7 +212,8 @@
           new OptionDefinition(ExecConstants.CPU_LOAD_AVERAGE),
           new OptionDefinition(ExecConstants.ENABLE_VECTOR_VALIDATOR),
           new OptionDefinition(ExecConstants.ENABLE_ITERATOR_VALIDATOR),
    -      new OptionDefinition(ExecConstants.OUTPUT_BATCH_SIZE_VALIDATOR, new 
OptionMetaData(OptionValue.AccessibleScopes.SYSTEM, true, false))
    +      new OptionDefinition(ExecConstants.OUTPUT_BATCH_SIZE_VALIDATOR, new 
OptionMetaData(OptionValue.AccessibleScopes.SYSTEM, true, false)),
    +      new 
OptionDefinition(ExecConstants.FRAG_RUNNER_RPC_TIMEOUT_VALIDATOR, new 
OptionMetaData(OptionValue.AccessibleScopes.SYSTEM, false, true)),
    --- End diff --
    
    internal should be true since we want this to show up in the internal 
options table and not the standard system options table. Changing to adminOnly 
= true seems reasonable.


---

Reply via email to