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