Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-1313

HdfsProxy changes from HDFS-481 missed in y20.1xx

    Details

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

      Description

      Some changes went missing between https://issues.apache.org/jira/secure/attachment/12441028/HDFS-481-bp-y20s.patch and https://issues.apache.org/jira/secure/attachment/12442210/HDFS-481-NEW.patch. The missing changes went into y20.100 but not into y20.1xx. This patch fixes it. Without this hdfsproxy does not work.

      1. HDFS-1313.y0.20.1xx_noprefix.patch
        5 kB
        Rohini Palaniswamy
      2. HDFS-1313.y0.20.1xx.patch
        5 kB
        Rohini Palaniswamy

        Issue Links

          Activity

          Rohini Palaniswamy created issue -
          Hide
          Rohini Palaniswamy added a comment -

          patch for y20.1xx

          Show
          Rohini Palaniswamy added a comment - patch for y20.1xx
          Rohini Palaniswamy made changes -
          Field Original Value New Value
          Attachment HDFS-1313.y0.20.1xx.patch [ 12450133 ]
          Tsz Wo Nicholas Sze made changes -
          Link This issue is related to HDFS-481 [ HDFS-481 ]
          Hide
          Tsz Wo Nicholas Sze added a comment -

          It seems that your patch is not simply copying codes from HDFS-481-NEW.patch but also contains some additional codes. Could you explain what are the additional codes for?

          Also, you need to use the --no-prefix option in git for generating patches.

          Show
          Tsz Wo Nicholas Sze added a comment - It seems that your patch is not simply copying codes from HDFS-481 -NEW.patch but also contains some additional codes. Could you explain what are the additional codes for? Also, you need to use the --no-prefix option in git for generating patches.
          Hide
          Rohini Palaniswamy added a comment -

          Attached patch with --no-prefix.

          Nicholas,
          HDFS-481 patch on StreamFile did not cleanly apply on y20.1.xx. So I had to port the changes. The additional change is build.xml where jetty jars are made part of the war file. Had to add it because reference to DataNode in StreamFile which has a HttpServer with lot of jetty dependencies caused ClassNotFoundException errors. I added it few weeks back for the internal builds, but forgot it was not part of the original patch.

          Show
          Rohini Palaniswamy added a comment - Attached patch with --no-prefix. Nicholas, HDFS-481 patch on StreamFile did not cleanly apply on y20.1.xx. So I had to port the changes. The additional change is build.xml where jetty jars are made part of the war file. Had to add it because reference to DataNode in StreamFile which has a HttpServer with lot of jetty dependencies caused ClassNotFoundException errors. I added it few weeks back for the internal builds, but forgot it was not part of the original patch.
          Rohini Palaniswamy made changes -
          Attachment HDFS-1313.y0.20.1xx_noprefix.patch [ 12450277 ]
          Hide
          Tsz Wo Nicholas Sze added a comment -

          +1 Patch looks good.

          Show
          Tsz Wo Nicholas Sze added a comment - +1 Patch looks good.
          Tsz Wo Nicholas Sze made changes -
          Hadoop Flags [Reviewed]
          Assignee Rohini Palaniswamy [ rohini ]
          Hide
          Rohini Palaniswamy added a comment -

          Changes checked into y20.1xx.

          Show
          Rohini Palaniswamy added a comment - Changes checked into y20.1xx.
          Rohini Palaniswamy made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Tom White made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          13d 1h 10m 1 Rohini Palaniswamy 04/Aug/10 08:44
          Resolved Resolved Closed Closed
          20d 13h 8m 1 Tom White 24/Aug/10 21:52

            People

            • Assignee:
              Rohini Palaniswamy
              Reporter:
              Rohini Palaniswamy
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development