Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-9723

Improve error message when hadoop archive output path already exists

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.0.4-alpha, 3.0.0-alpha1
    • Fix Version/s: 2.8.0, 3.0.0-alpha1
    • Component/s: None
    • Labels:
    • Hadoop Flags:
      Reviewed

      Description

      When creating a hadoop archive and specifying an output path of an already existing file, we get an "Invalid Output" error message.

      [schu@hdfs-vanilla-1 ~]$ hadoop archive -archiveName foo.har -p /user/schu testDir1 /user/schu
      Invalid Output: /user/schu/foo.har
      

      This error can be improved to tell users immediately that the output path already exists.

      1. HADOOP-9723.003.patch
        4 kB
        Yongjun Zhang
      2. HADOOP-9723.2.patch
        4 kB
        Akira Ajisaka
      3. HADOOP-9723.patch
        3 kB
        Jean-Baptiste Onofré

        Activity

        Hide
        jbonofre Jean-Baptiste Onofré added a comment -

        Attached patch for the trunk (applicable to hadoop-2 branch as well).

        Show
        jbonofre Jean-Baptiste Onofré added a comment - Attached patch for the trunk (applicable to hadoop-2 branch as well).
        Hide
        hadoopqa Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12592440/HADOOP-9723.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-tools/hadoop-archives.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2787//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2787//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12592440/HADOOP-9723.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-tools/hadoop-archives. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2787//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2787//console This message is automatically generated.
        Hide
        ajisakaa Akira Ajisaka added a comment -

        Here are two comments:

        1.

        +    if (outFs.exists(outputPath)) {
        +      throw new IOException("Archive path already exists");
        +    }
        +    if (outFs.isFile(dest)) {
        +      throw new IOException("Archive destination is a file");
        

        The following error messages seem to be better.

        "Archive path: " + outputPath + " already exists"
        "Archive destination: " + dest + " is a file"
        

        2.

        -import org.apache.hadoop.fs.FSDataInputStream;
        -import org.apache.hadoop.fs.FSDataOutputStream;
        -import org.apache.hadoop.fs.FileStatus;
        -import org.apache.hadoop.fs.FileSystem;
        -import org.apache.hadoop.fs.HarFileSystem;
        -import org.apache.hadoop.fs.Path;
        +import org.apache.hadoop.fs.*;
        

        To avoid gratuitous diff/merge problems, you don't need to turn imports into *.

        Show
        ajisakaa Akira Ajisaka added a comment - Here are two comments: 1. + if (outFs.exists(outputPath)) { + throw new IOException( "Archive path already exists" ); + } + if (outFs.isFile(dest)) { + throw new IOException( "Archive destination is a file" ); The following error messages seem to be better. "Archive path: " + outputPath + " already exists" "Archive destination: " + dest + " is a file" 2. - import org.apache.hadoop.fs.FSDataInputStream; - import org.apache.hadoop.fs.FSDataOutputStream; - import org.apache.hadoop.fs.FileStatus; - import org.apache.hadoop.fs.FileSystem; - import org.apache.hadoop.fs.HarFileSystem; - import org.apache.hadoop.fs.Path; + import org.apache.hadoop.fs.*; To avoid gratuitous diff/merge problems, you don't need to turn imports into *.
        Hide
        ajisakaa Akira Ajisaka added a comment -

        Attaching a patch to reflect the comments.

        Show
        ajisakaa Akira Ajisaka added a comment - Attaching a patch to reflect the comments.
        Hide
        hadoopqa Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12615287/HADOOP-9723.2.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-tools/hadoop-archives.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/3309//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/3309//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12615287/HADOOP-9723.2.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-tools/hadoop-archives. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/3309//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/3309//console This message is automatically generated.
        Hide
        hadoopqa Hadoop QA added a comment -



        +1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 15m 0s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 7m 57s There were no new javac warning messages.
        +1 javadoc 9m 55s There were no new javadoc warning messages.
        +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
        +1 checkstyle 0m 19s There were no new checkstyle issues.
        +1 whitespace 0m 0s The patch has no lines that end in whitespace.
        +1 install 1m 34s mvn install still works.
        +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse.
        +1 findbugs 0m 33s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
        +1 tools/hadoop tests 0m 50s Tests passed in hadoop-archives.
            37m 11s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12615287/HADOOP-9723.2.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / 6ae2a0d
        hadoop-archives test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6408/artifact/patchprocess/testrun_hadoop-archives.txt
        Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6408/testReport/
        Java 1.7.0_55
        uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6408/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 0s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 57s There were no new javac warning messages. +1 javadoc 9m 55s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 19s There were no new checkstyle issues. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 34s mvn install still works. +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse. +1 findbugs 0m 33s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 tools/hadoop tests 0m 50s Tests passed in hadoop-archives.     37m 11s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12615287/HADOOP-9723.2.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 6ae2a0d hadoop-archives test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6408/artifact/patchprocess/testrun_hadoop-archives.txt Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6408/testReport/ Java 1.7.0_55 uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6408/console This message was automatically generated.
        Hide
        nijel nijel added a comment -

        lgtm
        +1 for the change

        Show
        nijel nijel added a comment - lgtm +1 for the change
        Hide
        yzhangal Yongjun Zhang added a comment -

        Hi Guys, I'm reviewing this one.

        Show
        yzhangal Yongjun Zhang added a comment - Hi Guys, I'm reviewing this one.
        Hide
        yzhangal Yongjun Zhang added a comment -

        HI Akira Ajisaka,

        Thanks for your earlier patch, it looks good to me, except a few very minor thing.

        • Change "Destination directory xyz is a file" message to "Destination xyz should be a directory but is a file"; and change the test method accordingly
        • Use upper case in one exception "Source path ..." instead of "source path ..."
        • Change testExistingOutput to testOutputPathValidity,

        I'm in bug bash right now, to avoid another iteration, I'm uploading a revised version to address the above comments, hope you don't mind. Please comment on the change if you agree with the change.I will commit after you agree and jenkins test.

        Thanks.

        Show
        yzhangal Yongjun Zhang added a comment - HI Akira Ajisaka , Thanks for your earlier patch, it looks good to me, except a few very minor thing. Change "Destination directory xyz is a file" message to "Destination xyz should be a directory but is a file"; and change the test method accordingly Use upper case in one exception "Source path ..." instead of "source path ..." Change testExistingOutput to testOutputPathValidity, I'm in bug bash right now, to avoid another iteration, I'm uploading a revised version to address the above comments, hope you don't mind. Please comment on the change if you agree with the change.I will commit after you agree and jenkins test. Thanks.
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 15m 15s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 7m 44s There were no new javac warning messages.
        +1 javadoc 9m 48s There were no new javadoc warning messages.
        +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
        +1 checkstyle 0m 20s There were no new checkstyle issues.
        -1 whitespace 0m 1s The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix.
        +1 install 1m 39s mvn install still works.
        +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse.
        +1 findbugs 0m 33s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
        +1 tools/hadoop tests 0m 52s Tests passed in hadoop-archives.
            37m 13s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12731522/HADOOP-9723.003.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / 8f7c236
        whitespace https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/artifact/patchprocess/whitespace.txt
        hadoop-archives test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/artifact/patchprocess/testrun_hadoop-archives.txt
        Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/testReport/
        Java 1.7.0_55
        uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 15s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 44s There were no new javac warning messages. +1 javadoc 9m 48s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 20s There were no new checkstyle issues. -1 whitespace 0m 1s The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix. +1 install 1m 39s mvn install still works. +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse. +1 findbugs 0m 33s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 tools/hadoop tests 0m 52s Tests passed in hadoop-archives.     37m 13s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12731522/HADOOP-9723.003.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 8f7c236 whitespace https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/artifact/patchprocess/whitespace.txt hadoop-archives test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/artifact/patchprocess/testrun_hadoop-archives.txt Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/testReport/ Java 1.7.0_55 uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6550/console This message was automatically generated.
        Hide
        ajisakaa Akira Ajisaka added a comment -

        +1 for v3 patch. Thanks Yongjun Zhang for the update.

        Show
        ajisakaa Akira Ajisaka added a comment - +1 for v3 patch. Thanks Yongjun Zhang for the update.
        Hide
        ajisakaa Akira Ajisaka added a comment -

        Committed this to trunk and branch-2. Thanks Jean-Baptiste Onofré and Yongjun Zhang for contribution.

        Show
        ajisakaa Akira Ajisaka added a comment - Committed this to trunk and branch-2. Thanks Jean-Baptiste Onofré and Yongjun Zhang for contribution.
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-trunk-Commit #7813 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7813/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        • hadoop-common-project/hadoop-common/CHANGES.txt
        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7813 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7813/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java hadoop-common-project/hadoop-common/CHANGES.txt hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #195 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/195/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        • hadoop-common-project/hadoop-common/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #195 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/195/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java hadoop-common-project/hadoop-common/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Yarn-trunk #926 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/926/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-common-project/hadoop-common/CHANGES.txt
        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #926 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/926/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-common-project/hadoop-common/CHANGES.txt hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk #2124 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2124/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        • hadoop-common-project/hadoop-common/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2124 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2124/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java hadoop-common-project/hadoop-common/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #184 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/184/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        • hadoop-common-project/hadoop-common/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #184 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/184/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java hadoop-common-project/hadoop-common/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #194 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/194/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        • hadoop-common-project/hadoop-common/CHANGES.txt
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #194 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/194/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java hadoop-common-project/hadoop-common/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk #2142 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2142/)
        HADOOP-9723. Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5)

        • hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java
        • hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java
        • hadoop-common-project/hadoop-common/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2142 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2142/ ) HADOOP-9723 . Improve error message when hadoop archive output path already exists. Contributed by Jean-Baptiste Onofré and Yongjun Zhang. (aajisaka: rev 92c38e41e1fffb9d60d4fa5d4d2212777af9e9a5) hadoop-tools/hadoop-archives/src/test/java/org/apache/hadoop/tools/TestHadoopArchives.java hadoop-tools/hadoop-archives/src/main/java/org/apache/hadoop/tools/HadoopArchives.java hadoop-common-project/hadoop-common/CHANGES.txt
        Hide
        yzhangal Yongjun Zhang added a comment -

        Hi Akira Ajisaka,

        I just noticed that you committed this jira. Thanks for doing that and thanks for putting my name as one of the contributors, I really only did the review though

        Best regards.

        Show
        yzhangal Yongjun Zhang added a comment - Hi Akira Ajisaka , I just noticed that you committed this jira. Thanks for doing that and thanks for putting my name as one of the contributors, I really only did the review though Best regards.

          People

          • Assignee:
            yzhangal Yongjun Zhang
            Reporter:
            schu Stephen Chu
          • Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development