drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From paul-rogers <...@git.apache.org>
Subject [GitHub] drill pull request #983: DRILL-5819: Default value of security.admin.user_gr...
Date Wed, 11 Oct 2017 22:09:38 GMT
Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/983#discussion_r144149536
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClientFixture.java ---
    @@ -128,6 +128,21 @@ public void alterSystem(String key, Object value) {
       }
     
       /**
    +   * Reset a system option
    +   * @param key
    +   */
    +
    +  public void resetSystem(String key) {
    --- End diff --
    
    These are added in PR #970, in [this file](https://github.com/apache/drill/pull/970/files#diff-e2de7c62cedd56fbabbf3cff0f0663a2).
    
    But, this PR is likely to be committed before this one, so I'll deal with the merge conflict
later.


---

Mime
View raw message