Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-1036

in DelegationTokenFetch dfs.getURI returns no port

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.22.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      dfs.getUri().getPort() returns -1.

      1. fetchdt_doc.patch
        4 kB
        Boris Shkolnik
      2. HDFS-1036.patch
        6 kB
        Boris Shkolnik
      3. HDFS-1036-BP20.patch
        2 kB
        Boris Shkolnik
      4. HDFS-1036-BP20-1.patch
        5 kB
        Boris Shkolnik
      5. HDFS-1036-BP20-Fix.patch
        4 kB
        Boris Shkolnik
      6. HDFS-1036-doc.patch
        2 kB
        Boris Shkolnik

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #370 (See https://hudson.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/370/)

          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #370 (See https://hudson.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/370/ )
          Hide
          Boris Shkolnik added a comment -

          committed doc to trunk

          Show
          Boris Shkolnik added a comment - committed doc to trunk
          Hide
          Boris Shkolnik added a comment -

          doc update for trunk

          Show
          Boris Shkolnik added a comment - doc update for trunk
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #318 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/318/)
          HDFS-1036. in DelegationTokenFetch dfs.getURI returns no port

          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #318 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/318/ ) HDFS-1036 . in DelegationTokenFetch dfs.getURI returns no port
          Hide
          Boris Shkolnik added a comment -

          committed to trunk

          Show
          Boris Shkolnik added a comment - committed to trunk
          Hide
          Boris Shkolnik added a comment -

          ran test and test-patch. All tests passed.
          [exec]
          [exec]
          [exec] +1 overall.
          [exec]
          [exec] +1 @author. The patch does not contain any @author tags.
          [exec]
          [exec] +1 tests included. The patch appears to include 3 new or modified tests.
          [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.
          [exec]
          [exec]

          Show
          Boris Shkolnik added a comment - ran test and test-patch. All tests passed. [exec] [exec] [exec] +1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] +1 tests included. The patch appears to include 3 new or modified tests. [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. [exec] [exec]
          Hide
          Jitendra Nath Pandey added a comment -

          +1 for the patch.

          Show
          Jitendra Nath Pandey added a comment - +1 for the patch.
          Hide
          Boris Shkolnik added a comment -

          this is forrest docs update.

          Show
          Boris Shkolnik added a comment - this is forrest docs update.
          Hide
          Boris Shkolnik added a comment -

          addressed comment from Jakob, removing unneeded mock line..
          when(dfs.getUri()).thenReturn(uri);

          Show
          Boris Shkolnik added a comment - addressed comment from Jakob, removing unneeded mock line.. when(dfs.getUri()).thenReturn(uri);
          Hide
          Boris Shkolnik added a comment -

          fix for the patch. In case of Test configuration may be null.
          For the previous version. Not for commit.

          Show
          Boris Shkolnik added a comment - fix for the patch. In case of Test configuration may be null. For the previous version. Not for commit.
          Hide
          Jakob Homan added a comment -

          +1

          Show
          Jakob Homan added a comment - +1
          Hide
          Boris Shkolnik added a comment -

          for previous version, not for commit.

          Show
          Boris Shkolnik added a comment - for previous version, not for commit.

            People

            • Assignee:
              Boris Shkolnik
              Reporter:
              Boris Shkolnik
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development