[GitHub] drill pull request #923: DRILL-5723: Added System Internal Options That can ...

[ilooner]

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

    https://github.com/apache/drill/pull/923#discussion_r138171404
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/DrillConfigIterator.java
 ---
    @@ -58,17 +58,17 @@ public OptionValue next() {
           OptionValue optionValue = null;
           switch(cv.valueType()) {
           case BOOLEAN:
    -        optionValue = OptionValue.createBoolean(OptionType.BOOT, name, 
(Boolean) cv.unwrapped(), OptionScope.BOOT);
    +        optionValue = OptionValue.create(OptionType.BOOT, name, (Boolean) 
cv.unwrapped(), OptionScope.BOOT);
    --- End diff --
    
    Agreed


---