[ https://issues.apache.org/jira/browse/DRILL-5783?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16221210#comment-16221210 ]
ASF GitHub Bot commented on DRILL-5783: --------------------------------------- Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/984#discussion_r147261203 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -226,43 +217,16 @@ private void createConfig(FixtureBuilder builder) throws Exception { serviceSet = null; usesZk = true; - isLocal = false; } else { // Embedded Drillbit. serviceSet = RemoteServiceSet.getLocalServiceSet(); - isLocal = true; } } - private void startDrillbits(FixtureBuilder builder) throws Exception { -// // Ensure that Drill uses the log directory determined here rather than -// // it's hard-coded defaults. WIP: seems to be needed some times but -// // not others. -// -// String logDir = null; -// if (builder.tempDir != null) { -// logDir = builder.tempDir.getAbsolutePath(); -// } -// if (logDir == null) { -// logDir = config.getString(ExecConstants.DRILL_TMP_DIR); -// if (logDir != null) { -// logDir += "/drill/log"; -// } -// } -// if (logDir == null) { -// logDir = "/tmp/drill"; -// } -// new File(logDir).mkdirs(); -// System.setProperty("drill.log-dir", logDir); - - dfsTestTempDir = makeTempDir("dfs-test"); - - // Clean up any files that may have been left from the - // last run. - - preserveLocalFiles = builder.preserveLocalFiles; - removeLocalFiles(); + private void startDrillbits() throws Exception { + dfsTestTempDir = new File(getRootDir(), "dfs-test"); + dfsTestTempDir.mkdirs(); --- End diff -- This is possible through the BaseDirTestWatcher. It is configured through the BaseDireTestWatcher's constructor. If you pass it false it will not delete your directories at the end of each test. > Make code generation in the TopN operator more modular and test it > ------------------------------------------------------------------ > > Key: DRILL-5783 > URL: https://issues.apache.org/jira/browse/DRILL-5783 > Project: Apache Drill > Issue Type: Improvement > Reporter: Timothy Farkas > Assignee: Timothy Farkas > -- This message was sent by Atlassian JIRA (v6.4.14#64029)