[ https://issues.apache.org/jira/browse/DRILL-2100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15251938#comment-15251938 ]
ASF GitHub Bot commented on DRILL-2100: --------------------------------------- Github user vdiravka commented on a diff in the pull request: https://github.com/apache/drill/pull/454#discussion_r60585237 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.java --- @@ -223,7 +227,21 @@ public void close() { if (mSorter != null) { mSorter.clear(); } - + for(Iterator iter = this.currSpillDirs.iterator(); iter.hasNext(); iter.remove()) { + Path path = (Path)iter.next(); + try { + if (fs != null && path != null && fs.exists(path)) { + if (fs.delete(path, true)) { + fs.cancelDeleteOnExit(path); + } + } + } catch (IOException e) { + // since this is meant to be used in a batches's cleanup, we don't propagate the exception + final String message = "Unable to delete spill directory \"" + path + "\""; + System.err.println(message + '.'); --- End diff -- You are right. Updated > Drill not deleting spooling files > --------------------------------- > > Key: DRILL-2100 > URL: https://issues.apache.org/jira/browse/DRILL-2100 > Project: Apache Drill > Issue Type: Bug > Components: Execution - Relational Operators > Affects Versions: 0.8.0 > Reporter: Abhishek Girish > Assignee: Deneche A. Hakim > Fix For: 1.7.0 > > > Currently, after forcing queries to use an external sort by switching off > hash join/agg causes spill-to-disk files accumulating. > This causes issues with disk space availability when the spill is configured > to be on the local file system (/tmp/drill). Also not optimal when configured > to use DFS (custom). > Drill must clean up all temporary files created after a query completes or > after a drillbit restart. -- This message was sent by Atlassian JIRA (v6.3.4#6332)