[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_r138152185
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
 ---
    @@ -346,17 +347,63 @@ public void deleteAllOptions(OptionType type) {
         }
       }
     
    -  public void populateDefaultValues() {
    -
    +  public static CaseInsensitiveMap<OptionValue> 
populateDefualtValues(Map<String, OptionDefinition> definitions, DrillConfig 
bootConfig) {
    --- End diff --
    
    Done


---