Hadoop Common
  1. Hadoop Common
  2. HADOOP-6139

Incomplete help message is displayed for rm and rmr options.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.20.1
    • Fix Version/s: 0.20.1
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Tags:
      ygridqa

      Description

      If path/src is missing from rm or rmr option then help message is displayed for these options.

      On giving command "hadoop dfs -rm"
      help message displayed is "Usage: java FsShell [-rm <path>]"
      while it should be "Usage: java FsShell [-rm [-skipTrash] <path>]"

      Same issue is there with rmr option also.

      1. HADOOP-478-v20.patch
        1 kB
        Jakob Homan
      2. COMMON-478.patch
        1 kB
        Jakob Homan

        Issue Links

          Activity

          Hide
          Jakob Homan added a comment -

          Attaching patches for trunk and 20 branch. This is essentially a documentation change. Tested manually.

          Show
          Jakob Homan added a comment - Attaching patches for trunk and 20 branch. This is essentially a documentation change. Tested manually.
          Hide
          Jakob Homan added a comment -

          I had the issue moved to Common rather than HDFS since the affected file is FsShell.java, within the Common subproject.

          Show
          Jakob Homan added a comment - I had the issue moved to Common rather than HDFS since the affected file is FsShell.java, within the Common subproject.
          Hide
          Jakob Homan added a comment -

          submitting patch.

          Show
          Jakob Homan added a comment - submitting patch.
          Hide
          Jakob Homan added a comment -

          Ran unit tests locally. Passes all common unit tests.

          Show
          Jakob Homan added a comment - Ran unit tests locally. Passes all common unit tests.
          Hide
          Jakob Homan added a comment -

          test-patch:

               [exec] -1 overall.  
               [exec] 
               [exec]     +1 @author.  The patch does not contain any @author tags.
               [exec] 
               [exec]     -1 tests included.  The patch doesn't appear to include any new or modified tests.
               [exec]                         Please justify why no new tests are needed for this patch.
               [exec]                         Also please list what manual steps were performed to verify this patch.
               [exec] 
               [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
               [exec] 
               [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
               [exec] 
               [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
               [exec] 
               [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.

          No unit tests because is documentation change. Manual step as documented in jira description.

          Show
          Jakob Homan added a comment - test-patch: [exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no new tests are needed for this patch. [exec] Also please list what manual steps were performed to verify this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. No unit tests because is documentation change. Manual step as documented in jira description.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12413065/HADOOP-478-v20.patch
          against trunk revision 792812.

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

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          -1 patch. The patch command could not apply the patch.

          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/563/console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12413065/HADOOP-478-v20.patch against trunk revision 792812. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch. The patch command could not apply the patch. Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/563/console This message is automatically generated.
          Hide
          Tsz Wo Nicholas Sze added a comment -

          +1 patch looks good.

          Show
          Tsz Wo Nicholas Sze added a comment - +1 patch looks good.
          Hide
          Jakob Homan added a comment -

          Hudson tried to apply v20 patch, which is why I ran test-patch manually. Both pathces apply fine. Patch is ready to be committed.

          Show
          Jakob Homan added a comment - Hudson tried to apply v20 patch, which is why I ran test-patch manually. Both pathces apply fine. Patch is ready to be committed.
          Hide
          Tsz Wo Nicholas Sze added a comment -

          I have committed this. Thanks, Jakob!

          Show
          Tsz Wo Nicholas Sze added a comment - I have committed this. Thanks, Jakob!

            People

            • Assignee:
              Jakob Homan
              Reporter:
              Suman Sehgal
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development