Details

    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      There are 2 errors in section Test HttpFS is working in http://hadoop.apache.org/docs/r2.7.3/hadoop-hdfs-httpfs/ServerSetup.html

      ~ $ curl -i "http://<HTTPFSHOSTNAME>:14000?user.name=babu&op=homedir"
      HTTP/1.1 200 OK
      Content-Type: application/json
      Transfer-Encoding: chunked
      
      {"homeDir":"http:\/\/<HTTPFS_HOST>:14000\/user\/babu"}
      
      1. The URL path should be /webhdfs/v1.
      2. The op should be gethomedirectory, not homedir.

      The curl command would produce this error:

      $ curl 'http://localhost:14000/webhdfs/v1?op=homedir&user.name=hdfs' | json_pp
      {
         "RemoteException" : {
            "message" : "java.lang.IllegalArgumentException: No enum constant org.apache.hadoop.fs.http.client.HttpFSFileSystem.Operation.HOMEDIR",
            "exception" : "QueryParamException",
            "javaClassName" : "com.sun.jersey.api.ParamException$QueryParamException"
         }
      }
      

      The correct command should be:

      $ curl 'http://localhost:14000/webhdfs/v1?op=gethomedirectory&user.name=hdfs' | json_pp
      {
         "Path" : "/user/hdfs"
      }
      

        Activity

        Hide
        jzhuge John Zhuge added a comment -

        Patch 001:

        • Fix the HttpFS test command in ServerSetup.md.vm
        Show
        jzhuge John Zhuge added a comment - Patch 001: Fix the HttpFS test command in ServerSetup.md.vm
        Hide
        hadoopqa Hadoop QA added a comment -
        +1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 20s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 mvninstall 7m 16s trunk passed
        +1 mvnsite 0m 25s trunk passed
        +1 mvnsite 0m 23s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 asflicense 0m 15s The patch does not generate ASF License warnings.
        8m 54s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue HDFS-11174
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840453/HDFS-11174.001.patch
        Optional Tests asflicense mvnsite
        uname Linux e09c668de422 3.13.0-96-generic #143-Ubuntu SMP Mon Aug 29 20:15:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 01665e4
        modules C: hadoop-hdfs-project/hadoop-hdfs-httpfs U: hadoop-hdfs-project/hadoop-hdfs-httpfs
        Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17660/console
        Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 mvninstall 7m 16s trunk passed +1 mvnsite 0m 25s trunk passed +1 mvnsite 0m 23s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 asflicense 0m 15s The patch does not generate ASF License warnings. 8m 54s Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue HDFS-11174 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840453/HDFS-11174.001.patch Optional Tests asflicense mvnsite uname Linux e09c668de422 3.13.0-96-generic #143-Ubuntu SMP Mon Aug 29 20:15:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 01665e4 modules C: hadoop-hdfs-project/hadoop-hdfs-httpfs U: hadoop-hdfs-project/hadoop-hdfs-httpfs Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17660/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        yuanbo Yuanbo Liu added a comment -

        Tested this command locally. I'm +1(no-binding) for this patch.
        John Zhuge Thanks for your work.

        Show
        yuanbo Yuanbo Liu added a comment - Tested this command locally. I'm +1(no-binding) for this patch. John Zhuge Thanks for your work.
        Hide
        ajisakaa Akira Ajisaka added a comment -

        +1, checking this in.

        Show
        ajisakaa Akira Ajisaka added a comment - +1, checking this in.
        Hide
        ajisakaa Akira Ajisaka added a comment -

        Committed this to trunk, branch-2, branch-2.8, and branch-2.7. Thanks John Zhuge for the contribution and thanks Yuanbo Liu for the review.

        Show
        ajisakaa Akira Ajisaka added a comment - Committed this to trunk, branch-2, branch-2.8, and branch-2.7. Thanks John Zhuge for the contribution and thanks Yuanbo Liu for the review.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10892 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10892/)
        HDFS-11174. Wrong HttpFS test command in doc. Contributed by John Zhuge. (aajisaka: rev bb1b5e083b6f32646412b292d061ce5721f5dbe1)

        • (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/site/markdown/ServerSetup.md.vm
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10892 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10892/ ) HDFS-11174 . Wrong HttpFS test command in doc. Contributed by John Zhuge. (aajisaka: rev bb1b5e083b6f32646412b292d061ce5721f5dbe1) (edit) hadoop-hdfs-project/hadoop-hdfs-httpfs/src/site/markdown/ServerSetup.md.vm
        Hide
        jzhuge John Zhuge added a comment -
        Show
        jzhuge John Zhuge added a comment - Thanks Akira Ajisaka and Yuanbo Liu !

          People

          • Assignee:
            jzhuge John Zhuge
            Reporter:
            jzhuge John Zhuge
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development