Github user vdiravka commented on a diff in the pull request:
https://github.com/apache/drill/pull/1083#discussion_r160666020
--- 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 --
Agree.
I have changed it. Empty directory is created in the scope of one test. But
to avoid duplicating code, when in the class there are several test cases with
using empty dir, the last is created once for entire class.
---
