Yida Wu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20852
Change subject: IMPALA-12681: Release file descriptors for partial written temporary files ...................................................................... IMPALA-12681: Release file descriptors for partial written temporary files The patch fixes a bug that the partial written temporary files are removed without releasing the file descriptors. This patch fixes the bug by adding a call to Close() during the destruction of the LocalFileWriter class, ensuring proper release of the file handle. Additionally, the patch introduces the immediate release of the file handle by adding the file writer reset when the Delete() is called within the DiskFile class. Testing: Manual testing of remote spilling, with the patch that there are no references to the deleted files in the /proc/x/fd/. Conversely, in the absence of the patch, such references are observed. Change-Id: I58a2bac419ced806d6f5a32bcdf24d79e078ab14 --- M be/src/runtime/io/disk-file.cc M be/src/runtime/io/local-file-writer.cc M be/src/runtime/io/local-file-writer.h 3 files changed, 13 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/52/20852/1 -- To view, visit http://gerrit.cloudera.org:8080/20852 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I58a2bac419ced806d6f5a32bcdf24d79e078ab14 Gerrit-Change-Number: 20852 Gerrit-PatchSet: 1 Gerrit-Owner: Yida Wu <wydbaggio...@gmail.com>
