-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDFS-15557. Log the reason why a storage log file can't be deleted #2274
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
try { | ||
Files.delete(origFile.toPath()); | ||
} catch (IOException e) { | ||
throw new IOException("Could not delete original file " + origFile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it simpler
throw new IOException("Could not delete original file " + origFile, e);
Other than that, +1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. Thanks.
💔 -1 overall
This message was automatically generated. |
Not sure why the build came out so badly... let's see if we can retrigger. |
@liuml07 any further comments? |
+1 Not sure why the QA is not coming back cleanly here...not related to the change. |
…pache#2274) (cherry picked from commit ae089f2) Change-Id: I59e5a3508fa630b327a5d0d573a2cce9a1ed3570
https://issues.apache.org/jira/browse/HDFS-15557
NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute