[
https://issues.apache.org/jira/browse/HDFS-13133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16451171#comment-16451171
]
Hudson commented on HDFS-13133:
-------------------------------
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14057 (See
[https://builds.apache.org/job/Hadoop-trunk-Commit/14057/])
HDFS-13133. Ozone: OzoneFileSystem: Calling delete with non-existing (msingh:
rev f3d07efac1a7007ed0493486a6aff26cbaa09b22)
* (edit)
hadoop-tools/hadoop-ozone/src/main/java/org/apache/hadoop/fs/ozone/OzoneFileSystem.java
> Ozone: OzoneFileSystem: Calling delete with non-existing path shouldn't be
> logged on ERROR level
> ------------------------------------------------------------------------------------------------
>
> Key: HDFS-13133
> URL: https://issues.apache.org/jira/browse/HDFS-13133
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: ozone
> Affects Versions: HDFS-7240
> Reporter: Elek, Marton
> Assignee: Elek, Marton
> Priority: Major
> Fix For: HDFS-7240
>
> Attachments: HDFS-13133-HDFS-7240.001.patch
>
>
> During the test of OzoneFileSystem with spark I noticed ERROR messages
> multiple times:
> Something like this:
> {code}
> 2018-02-11 15:54:54 ERROR OzoneFileSystem:409 - Couldn't delete
> o3://bucket1.test/user/hadoop/.sparkStaging/application_1518349702045_0008 -
> does not exist
> {code}
> I checked the other implemetations, and they use DEBUG level. I think it's
> expected that the path sometimes points to a non-existing dir/file.
> To be consistent with the other implemetation I propose to lower the log
> level to debug.
> Examples from other file systems:
> S3AFileSystem:
> {code}
> } catch (FileNotFoundException e) {
> LOG.debug("Couldn't delete {} - does not exist", f);
> instrumentation.errorIgnored();
> return false;
> } catch (AmazonClientException e) {
> throw translateException("delete", f, e);
> }
> {code}
> Alyun:
> {code}
> try {
> return innerDelete(getFileStatus(path), recursive);
> } catch (FileNotFoundException e) {
> LOG.debug("Couldn't delete {} - does not exist", path);
> return false;
> }
> {code}
> SFTP:
> {code}
> } catch (FileNotFoundException e) {
> // file not found, no need to delete, return true
> return false;
> }
> {code}
> SwiftNativeFileSystem:
> {code}
> try {
> return store.delete(path, recursive);
> } catch (FileNotFoundException e) {
> //base path was not found.
> return false;
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
