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: MD-2769: DRILL-5819: Default value of security.admi...
Date Wed, 11 Oct 2017 04:40:22 GMT
Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/983#discussion_r143910606
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java
---
    @@ -115,4 +124,59 @@ private void setOptHelper() throws Exception {
           test(String.format("ALTER SYSTEM SET `%s` = %d;", ExecConstants.SLICE_TARGET, ExecConstants.SLICE_TARGET_DEFAULT));
         }
       }
    +
    +  @Test
    +  public void testAdminUserOptions() throws Exception {
    +    FixtureBuilder builder = ClusterFixture.builder();
    +
    +    try (ClusterFixture cluster = builder.build();
    --- End diff --
    
    No big deal, but if you set no options, then a shorthand is:
    
    ```
    cluster = ClusterFixture.standardCluster();
    ```


---

Mime
View raw message