Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-3698

TestHftpFileSystem is failing in branch-1 due to changed default secure port

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0, 1.2.0
    • Fix Version/s: 1.1.0, 1.2.0
    • Component/s: security
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      This test is failing since the default secure port changed to the HTTP port upon the commit of HDFS-2617.

      1. HDFS-3698.patch
        3 kB
        Aaron T. Myers

        Issue Links

          Activity

          Aaron T. Myers created issue -
          Aaron T. Myers made changes -
          Field Original Value New Value
          Link This issue is broken by HDFS-2617 [ HDFS-2617 ]
          Hide
          Aaron T. Myers added a comment -

          Here's a patch which addresses the issue. Mostly this is just changing the test case, but I also discovered a potential infinite recursion in the case of HSFTP. This patch fixes that as well.

          I tested this patch by ensuring that TestHftpFileSystem passes on branch-1 with this patch applied.

          Show
          Aaron T. Myers added a comment - Here's a patch which addresses the issue. Mostly this is just changing the test case, but I also discovered a potential infinite recursion in the case of HSFTP. This patch fixes that as well. I tested this patch by ensuring that TestHftpFileSystem passes on branch-1 with this patch applied.
          Aaron T. Myers made changes -
          Attachment HDFS-3698.patch [ 12537442 ]
          Aaron T. Myers made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12537442/HDFS-3698.patch
          against trunk revision .

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

          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2883//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/12537442/HDFS-3698.patch against trunk revision . -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/2883//console This message is automatically generated.
          Hide
          Todd Lipcon added a comment -

          Looks good to me. +1

          Show
          Todd Lipcon added a comment - Looks good to me. +1
          Hide
          Eli Collins added a comment -

          +1 lgtm as well

          Show
          Eli Collins added a comment - +1 lgtm as well
          Hide
          Aaron T. Myers added a comment -

          Thanks a lot for the reviews, Eli and Todd. I've just committed this to branch-1.

          Show
          Aaron T. Myers added a comment - Thanks a lot for the reviews, Eli and Todd. I've just committed this to branch-1.
          Aaron T. Myers made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Fix Version/s 1.2.0 [ 12321657 ]
          Resolution Fixed [ 1 ]
          Hide
          Arpit Gupta added a comment -

          reopening as the same issue is seen in branch-1.1

          Show
          Arpit Gupta added a comment - reopening as the same issue is seen in branch-1.1
          Arpit Gupta made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Arpit Gupta made changes -
          Affects Version/s 1.1.0 [ 12317959 ]
          Suresh Srinivas made changes -
          Fix Version/s 1.1.0 [ 12317959 ]
          Hide
          Suresh Srinivas added a comment -

          I merged the patch to branch 1.1

          Show
          Suresh Srinivas added a comment - I merged the patch to branch 1.1
          Suresh Srinivas made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Aaron T. Myers added a comment -

          Thanks guys.

          Show
          Aaron T. Myers added a comment - Thanks guys.
          Hide
          Matt Foley added a comment -

          Closed upon release of Hadoop-1.1.0.

          Show
          Matt Foley added a comment - Closed upon release of Hadoop-1.1.0.
          Matt Foley made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Aaron T. Myers
              Reporter:
              Aaron T. Myers
            • Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development