Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-7888

Change DataStreamer/DFSOutputStream/DFSPacket for convenience of subclassing

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.8.0, 3.0.0-alpha1
    • Component/s: hdfs-client
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      HDFS-7793 refactors class DFSOutputStream on trunk which makes DFSOutputStream a class without any inner classes. We want to subclass DFSOutputStream to support striping layout writing. This JIRA depends upon HDFS-7793 and tries to change DataStreamer/DFSOutputStream/DFSPacket for convenience of subclassing.

      1. HDFS-7888-001.patch
        23 kB
        Li Bo
      2. HDFS-7888-trunk-001.patch
        23 kB
        Li Bo
      3. HDFS-7888-trunk-002.patch
        11 kB
        Zhe Zhang
      4. HDFS-7888-trunk-003.patch
        11 kB
        Zhe Zhang
      5. HDFS-7888-trunk-004.patch
        10 kB
        Zhe Zhang

        Issue Links

          Activity

          Hide
          libo-intel Li Bo added a comment -

          This JIRA can be a part of HDFS-7889. One reason to create this JIRA is to reduce the patch size of HDFS-7889 for the convenience of review.

          Show
          libo-intel Li Bo added a comment - This JIRA can be a part of HDFS-7889 . One reason to create this JIRA is to reduce the patch size of HDFS-7889 for the convenience of review.
          Hide
          szetszwo Tsz Wo Nicholas Sze added a comment -

          Should we do this in trunk?

          Show
          szetszwo Tsz Wo Nicholas Sze added a comment - Should we do this in trunk?
          Hide
          libo-intel Li Bo added a comment -

          hi Nicholas
          I have just uploaded a patch to HDFS-7889. If you think that patch is big and not easy to review, I can split it into two parts, one patch for this jira and the other for HDFS-7889; otherwise I will mark this jiar as duplicated with HDFS-7889.

          Show
          libo-intel Li Bo added a comment - hi Nicholas I have just uploaded a patch to HDFS-7889 . If you think that patch is big and not easy to review, I can split it into two parts, one patch for this jira and the other for HDFS-7889 ; otherwise I will mark this jiar as duplicated with HDFS-7889 .
          Hide
          szetszwo Tsz Wo Nicholas Sze added a comment -

          Let's split the patch since the code change described here should be committed to trunk and the new code for DFSStripedOutputStream should be committed to the 7285 branch.

          Show
          szetszwo Tsz Wo Nicholas Sze added a comment - Let's split the patch since the code change described here should be committed to trunk and the new code for DFSStripedOutputStream should be committed to the 7285 branch.
          Hide
          libo-intel Li Bo added a comment -

          About patch 001 :
          1. Change some fields and methods from private to protected
          2. streamer field used to be accessed directly; now added a getter getStreamer()
          3. Refactor: separate out 2 methods in writeChunkImpl: adjustChunkBoundary and encounterBlockBoundary

          Please note this patch is generated based on 7285 branch.

          Show
          libo-intel Li Bo added a comment - About patch 001 : 1. Change some fields and methods from private to protected 2. streamer field used to be accessed directly; now added a getter getStreamer() 3. Refactor: separate out 2 methods in writeChunkImpl: adjustChunkBoundary and encounterBlockBoundary Please note this patch is generated based on 7285 branch.
          Hide
          zhz Zhe Zhang added a comment -

          Thanks Bo! The patch looks good to me. All 3 changes are pretty clear and straightforward. A few nits:

          1. An unnecessary change in import:
             import java.util.concurrent.atomic.AtomicReference;
            -
            
          2. Looks like the following line is too long?
            +        dfsClient.getPathTraceScope(className.substring(0, className.lastIndexOf('.') + 1)
            

          Could you also post a trunk-based patch to trigger Jenkins? Thanks!

          Show
          zhz Zhe Zhang added a comment - Thanks Bo! The patch looks good to me. All 3 changes are pretty clear and straightforward. A few nits: An unnecessary change in import: import java.util.concurrent.atomic.AtomicReference; - Looks like the following line is too long? + dfsClient.getPathTraceScope(className.substring(0, className.lastIndexOf('.') + 1) Could you also post a trunk-based patch to trigger Jenkins? Thanks!
          Hide
          libo-intel Li Bo added a comment -

          HDFS-7888-trunk-001.patch fixes the two problems.

          Show
          libo-intel Li Bo added a comment - HDFS-7888 -trunk-001.patch fixes the two problems.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12708333/HDFS-7888-trunk-001.patch
          against trunk revision 85dc3c1.

          +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 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10124//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10124//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12708333/HDFS-7888-trunk-001.patch against trunk revision 85dc3c1. +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 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10124//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10124//console This message is automatically generated.
          Hide
          zhz Zhe Zhang added a comment -

          Thanks Bo for the trunk patch!

          After reading it again I realized we can further shrink the change by keeping the streamer variable. In DFSStripedOutputStream we can update the value of streamer when we switch to the next streamer. I'm attaching a smaller patch with just steps #1 and #3 in the summary .

          TestDatanodeManager is unrelated and passes locally.

          Show
          zhz Zhe Zhang added a comment - Thanks Bo for the trunk patch! After reading it again I realized we can further shrink the change by keeping the streamer variable. In DFSStripedOutputStream we can update the value of streamer when we switch to the next streamer. I'm attaching a smaller patch with just steps #1 and #3 in the summary . TestDatanodeManager is unrelated and passes locally.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12708465/HDFS-7888-trunk-002.patch
          against trunk revision 79f7f2a.

          +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 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.namenode.TestCheckpoint

          The following test timeouts occurred in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10130//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10130//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12708465/HDFS-7888-trunk-002.patch against trunk revision 79f7f2a. +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 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.namenode.TestCheckpoint The following test timeouts occurred in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10130//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10130//console This message is automatically generated.
          Hide
          libo-intel Li Bo added a comment -

          That's a very good improvement of the patch. I will also update the patch of HDFS-7889

          Show
          libo-intel Li Bo added a comment - That's a very good improvement of the patch. I will also update the patch of HDFS-7889
          Hide
          zhz Zhe Zhang added a comment -

          The 2 failed tests are unrelated and pass locally.

          Show
          zhz Zhe Zhang added a comment - The 2 failed tests are unrelated and pass locally.
          Hide
          jingzhao Jing Zhao added a comment -

          Thanks for working on this, Bo. The patch looks good to me. Some minors:

          1. how about using getSimpleName here?
            +    String path = this.getClass().getName();
            +    String className = path.substring(path.lastIndexOf('.') + 1);
                 TraceScope scope =
            -        dfsClient.getPathTraceScope("DFSOutputStream#writeChunk", src);
            +        dfsClient.getPathTraceScope(className + "#writeChunk", src);
            
          2. encounterBlockBoundary may not be an appropriate method name. maybe we can rename it to something like endBlock?
          Show
          jingzhao Jing Zhao added a comment - Thanks for working on this, Bo. The patch looks good to me. Some minors: how about using getSimpleName here? + String path = this .getClass().getName(); + String className = path.substring(path.lastIndexOf('.') + 1); TraceScope scope = - dfsClient.getPathTraceScope( "DFSOutputStream#writeChunk" , src); + dfsClient.getPathTraceScope(className + "#writeChunk" , src); encounterBlockBoundary may not be an appropriate method name. maybe we can rename it to something like endBlock ?
          Hide
          zhz Zhe Zhang added a comment -

          Thanks Jing for the review! These 2 changes sound good to me. Since they are simple I just directly updated the patch to save time.

          Show
          zhz Zhe Zhang added a comment - Thanks Jing for the review! These 2 changes sound good to me. Since they are simple I just directly updated the patch to save time.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12708772/HDFS-7888-trunk-003.patch
          against trunk revision 796fb26.

          +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 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The following test timeouts occurred in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10148//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10148//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12708772/HDFS-7888-trunk-003.patch against trunk revision 796fb26. +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 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The following test timeouts occurred in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10148//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10148//console This message is automatically generated.
          Hide
          szetszwo Tsz Wo Nicholas Sze added a comment -

          +1 patch looks good.

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

          Unfortunately, the patch does not apply anymore. Could you update it?

          Show
          szetszwo Tsz Wo Nicholas Sze added a comment - Unfortunately, the patch does not apply anymore. Could you update it?
          Hide
          zhz Zhe Zhang added a comment -

          Thanks Nicholas for the review!

          Rebased to resolve conflict with HDFS-8026.

          Show
          zhz Zhe Zhang added a comment - Thanks Nicholas for the review! Rebased to resolve conflict with HDFS-8026 .
          Hide
          libo-intel Li Bo added a comment -

          The changes also look good to me. Thanks for the review of Zhe, Jing and Nicholas.

          Show
          libo-intel Li Bo added a comment - The changes also look good to me. Thanks for the review of Zhe, Jing and Nicholas.
          Hide
          hadoopqa Hadoop QA added a comment -

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

          +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 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager
          org.apache.hadoop.hdfs.web.TestWebHDFS

          The following test timeouts occurred in hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.TestRollingUpgrade

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10158//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10158//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12708865/HDFS-7888-trunk-004.patch against trunk revision f383fd9. +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 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.blockmanagement.TestDatanodeManager org.apache.hadoop.hdfs.web.TestWebHDFS The following test timeouts occurred in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.TestRollingUpgrade Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/10158//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/10158//console This message is automatically generated.
          Hide
          zhz Zhe Zhang added a comment -

          The 3 tests are unrelated to the patch, and all pass locally.

          Apache git server is down at this moment though..

          Show
          zhz Zhe Zhang added a comment - The 3 tests are unrelated to the patch, and all pass locally. Apache git server is down at this moment though..
          Hide
          szetszwo Tsz Wo Nicholas Sze added a comment -

          +1 the new patch looks good.

          Show
          szetszwo Tsz Wo Nicholas Sze added a comment - +1 the new patch looks good.
          Hide
          szetszwo Tsz Wo Nicholas Sze added a comment -

          I have committed this. Thanks Li!

          Show
          szetszwo Tsz Wo Nicholas Sze added a comment - I have committed this. Thanks Li!
          Hide
          zhz Zhe Zhang added a comment -

          Thanks Li Bo for the patch and reviews from Jing and Nicholas! I just rebased HDFS-7285 branch to include it.

          Show
          zhz Zhe Zhang added a comment - Thanks Li Bo for the patch and reviews from Jing and Nicholas! I just rebased HDFS-7285 branch to include it.
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #7493 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7493/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7493 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7493/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #152 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/152/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #152 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/152/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #886 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/886/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #886 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/886/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #143 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/143/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #143 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/143/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk #2084 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2084/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2084 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2084/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #153 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/153/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #153 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/153/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #2102 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2102/)
          HDFS-7888. Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a)

          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java
          • hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2102 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2102/ ) HDFS-7888 . Change DFSOutputStream and DataStreamer for convenience of subclassing. Contributed by Li Bo (szetszwo: rev 9ed43f2189fb4674b7379e8e995d53d4970d5c3a) hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DataStreamer.java hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java

            People

            • Assignee:
              libo-intel Li Bo
              Reporter:
              libo-intel Li Bo
            • Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development