Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16294
Change subject: IMPALA-10044: Fix cleanup for bootstrap_toolchain.py failure case ...................................................................... IMPALA-10044: Fix cleanup for bootstrap_toolchain.py failure case If DownloadUnpackTarball::download()'s wget_and_unpack_package call hits an exception, the exception handler cleans up any created directories. Currently, it erroneously cleans up the directory where the tarballs are downloaded even when it is not a temporary directory. This would delete the entire toolchain. This fixes the cleanup to only delete that directory if it is a temporary directory. Testing: - Simulated exception from wget_and_unpack_package and verified behavior. Change-Id: Ia57f56b6717635af94247fce50b955c07a57d113 --- M bin/bootstrap_toolchain.py 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/16294/1 -- To view, visit http://gerrit.cloudera.org:8080/16294 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ia57f56b6717635af94247fce50b955c07a57d113 Gerrit-Change-Number: 16294 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]>
