drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From arina-ielchiieva <...@git.apache.org>
Subject [GitHub] drill pull request #1083: DRILL-4185: UNION ALL involving empty directory on...
Date Tue, 09 Jan 2018 21:26:15 GMT
Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1083#discussion_r160525303
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/BaseTestQuery.java ---
    @@ -119,6 +125,15 @@ public static void setupDefaultTestCluster() throws Exception {
         // turns on the verbose errors in tests
         // sever side stacktraces are added to the message before sending back to the client
         test("ALTER SESSION SET `exec.errors.verbose` = true");
    +    emptyDirCreating();
    --- End diff --
    
    Why do we create empty directory for all tests that extend this class? I guess need to
create it only for those tests that need it.


---

Mime
View raw message