Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.0
    • Fix Version/s: 2.8.0, 3.0.0-alpha2
    • Component/s: fs/s3
    • Labels:
      None
    • Target Version/s:

      Description

      The fs.s3a.readahead.range value is measured in bytes, but can be hundreds of KB. Easier to use getLongBytes and set to things like "300k"

      This will be backwards compatible with the existing settings if anyone is using them, because the no-prefix default will still be bytes

      1. HADOOP-13680-branch-2-004.patch
        10 kB
        Steve Loughran
      2. HADOOP-13680.001.patch
        7 kB
        Abhishek Modi

        Issue Links

          Activity

          Hide
          abmodi Abhishek Modi added a comment -

          Steve Loughran I would like to work on this issue. I have already created a patch for the same. Thanks for your time.

          Show
          abmodi Abhishek Modi added a comment - Steve Loughran I would like to work on this issue. I have already created a patch for the same. Thanks for your time.
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user abmodi opened a pull request:

          https://github.com/apache/hadoop/pull/144

          HADOOP-13680. fs.s3a.readahead.range to use getLongBytes

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/abmodi/hadoop HADOOP-13680

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/hadoop/pull/144.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #144


          commit 5967565190331b361a9c0f025b20f82f4e2014ce
          Author: Abhishek Modi <abmodi90@gmail.com>
          Date: 2016-10-23T14:32:28Z

          HADOOP-13680. fs.s3a.readahead.range to use getLongBytes


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user abmodi opened a pull request: https://github.com/apache/hadoop/pull/144 HADOOP-13680 . fs.s3a.readahead.range to use getLongBytes You can merge this pull request into a Git repository by running: $ git pull https://github.com/abmodi/hadoop HADOOP-13680 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/hadoop/pull/144.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #144 commit 5967565190331b361a9c0f025b20f82f4e2014ce Author: Abhishek Modi <abmodi90@gmail.com> Date: 2016-10-23T14:32:28Z HADOOP-13680 . fs.s3a.readahead.range to use getLongBytes
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user steveloughran opened a pull request:

          https://github.com/apache/hadoop/pull/145

          Steve's patch for '13680 get long bytes

          This for HADOOP-13680; just naming it so that yetus does not give it priority over the other PR

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/steveloughran/hadoop s3/HADOOP-13680-get-long-bytes

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/hadoop/pull/145.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #145


          commit 681e412c7d59e5bc83702b2dc3715da8054b02f4
          Author: Steve Loughran <stevel@apache.org>
          Date: 2016-10-24T09:58:47Z

          HADOOP-13680 get long bytes


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user steveloughran opened a pull request: https://github.com/apache/hadoop/pull/145 Steve's patch for '13680 get long bytes This for HADOOP-13680 ; just naming it so that yetus does not give it priority over the other PR You can merge this pull request into a Git repository by running: $ git pull https://github.com/steveloughran/hadoop s3/ HADOOP-13680 -get-long-bytes Alternatively you can review and apply these changes as the patch at: https://github.com/apache/hadoop/pull/145.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #145 commit 681e412c7d59e5bc83702b2dc3715da8054b02f4 Author: Steve Loughran <stevel@apache.org> Date: 2016-10-24T09:58:47Z HADOOP-13680 get long bytes
          Hide
          stevel@apache.org Steve Loughran added a comment -

          Ooh I'd forgotten about this; I'd one a quick PoC patch and then stashed it off, while I was doing HADOOP-13560. I've applied that to branch-2 and stuck it up https://github.com/steveloughran/hadoop/tree/s3/HADOOP-13680-get-long-bytes. I don't want to attach the patch or link it as a PR here as it will only cause confusion.

          Here it is https://github.com/apache/hadoop/pull/145

          Comparing the two:

          • I added longBytesOption to S3AUtils for broader use; by using it in getMultipartSizeProperty. then S3ABlockOutputStream gets it for its sizes too
          • we both patched S3AFileSystem
          • you added a test, which makes it vastly superior.
          • neither of us documented this.

          How about we combine them: my changes to s3a Test utils, your changes on s3a FS and the test, and the docs get updated to indicated that byte values can be declared (we can do this in the examples and core-default for completeness);

          ...if you do that, I could probably stay untainted enough to be the reviewer....

          Show
          stevel@apache.org Steve Loughran added a comment - Ooh I'd forgotten about this; I'd one a quick PoC patch and then stashed it off, while I was doing HADOOP-13560 . I've applied that to branch-2 and stuck it up https://github.com/steveloughran/hadoop/tree/s3/HADOOP-13680-get-long-bytes . I don't want to attach the patch or link it as a PR here as it will only cause confusion. Here it is https://github.com/apache/hadoop/pull/145 Comparing the two: I added longBytesOption to S3AUtils for broader use; by using it in getMultipartSizeProperty . then S3ABlockOutputStream gets it for its sizes too we both patched S3AFileSystem you added a test, which makes it vastly superior. neither of us documented this. How about we combine them: my changes to s3a Test utils, your changes on s3a FS and the test, and the docs get updated to indicated that byte values can be declared (we can do this in the examples and core-default for completeness); ...if you do that, I could probably stay untainted enough to be the reviewer....
          Hide
          abmodi Abhishek Modi added a comment -

          Sure. I will do this and update the pull request. Thanks for your inputs.

          Show
          abmodi Abhishek Modi added a comment - Sure. I will do this and update the pull request. Thanks for your inputs.
          Hide
          stevel@apache.org Steve Loughran added a comment -

          oh, thank you for getting your hands dirty in the s3a work. We've a lot to do...

          Show
          stevel@apache.org Steve Loughran added a comment - oh, thank you for getting your hands dirty in the s3a work. We've a lot to do...
          Hide
          abmodi Abhishek Modi added a comment -

          I have updated the PR with suggested changes. Steve Loughran Could you please take a look.

          Show
          abmodi Abhishek Modi added a comment - I have updated the PR with suggested changes. Steve Loughran Could you please take a look.
          Hide
          abmodi Abhishek Modi added a comment -

          I was able to run S3A tests successfully with these changes.

          Show
          abmodi Abhishek Modi added a comment - I was able to run S3A tests successfully with these changes.
          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 21s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 test4tests 0m 0s 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 mvninstall 6m 37s branch-2 passed
          +1 compile 0m 17s branch-2 passed with JDK v1.8.0_101
          +1 compile 0m 19s branch-2 passed with JDK v1.7.0_111
          +1 checkstyle 0m 15s branch-2 passed
          +1 mvnsite 0m 24s branch-2 passed
          +1 mvneclipse 0m 15s branch-2 passed
          +1 findbugs 0m 34s branch-2 passed
          +1 javadoc 0m 13s branch-2 passed with JDK v1.8.0_101
          +1 javadoc 0m 15s branch-2 passed with JDK v1.7.0_111
          +1 mvninstall 0m 17s the patch passed
          +1 compile 0m 14s the patch passed with JDK v1.8.0_101
          +1 javac 0m 14s the patch passed
          +1 compile 0m 16s the patch passed with JDK v1.7.0_111
          +1 javac 0m 16s the patch passed
          -0 checkstyle 0m 11s hadoop-tools/hadoop-aws: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
          +1 mvnsite 0m 21s the patch passed
          +1 mvneclipse 0m 11s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 0m 42s the patch passed
          +1 javadoc 0m 10s the patch passed with JDK v1.8.0_101
          +1 javadoc 0m 13s the patch passed with JDK v1.7.0_111
          +1 unit 0m 21s hadoop-aws in the patch passed with JDK v1.7.0_111.
          +1 asflicense 0m 16s The patch does not generate ASF License warnings.
          14m 46s



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:b59b8b7
          JIRA Issue HADOOP-13680
          GITHUB PR https://github.com/apache/hadoop/pull/145
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 5cf99e50f144 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision branch-2 / ea9a1be
          Default Java 1.7.0_111
          Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_101 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_111
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-HADOOP-Build/10916/artifact/patchprocess/diff-checkstyle-hadoop-tools_hadoop-aws.txt
          JDK v1.7.0_111 Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/10916/testReport/
          modules C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/10916/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 21s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. -1 test4tests 0m 0s 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 mvninstall 6m 37s branch-2 passed +1 compile 0m 17s branch-2 passed with JDK v1.8.0_101 +1 compile 0m 19s branch-2 passed with JDK v1.7.0_111 +1 checkstyle 0m 15s branch-2 passed +1 mvnsite 0m 24s branch-2 passed +1 mvneclipse 0m 15s branch-2 passed +1 findbugs 0m 34s branch-2 passed +1 javadoc 0m 13s branch-2 passed with JDK v1.8.0_101 +1 javadoc 0m 15s branch-2 passed with JDK v1.7.0_111 +1 mvninstall 0m 17s the patch passed +1 compile 0m 14s the patch passed with JDK v1.8.0_101 +1 javac 0m 14s the patch passed +1 compile 0m 16s the patch passed with JDK v1.7.0_111 +1 javac 0m 16s the patch passed -0 checkstyle 0m 11s hadoop-tools/hadoop-aws: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2) +1 mvnsite 0m 21s the patch passed +1 mvneclipse 0m 11s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 0m 42s the patch passed +1 javadoc 0m 10s the patch passed with JDK v1.8.0_101 +1 javadoc 0m 13s the patch passed with JDK v1.7.0_111 +1 unit 0m 21s hadoop-aws in the patch passed with JDK v1.7.0_111. +1 asflicense 0m 16s The patch does not generate ASF License warnings. 14m 46s Subsystem Report/Notes Docker Image:yetus/hadoop:b59b8b7 JIRA Issue HADOOP-13680 GITHUB PR https://github.com/apache/hadoop/pull/145 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 5cf99e50f144 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision branch-2 / ea9a1be Default Java 1.7.0_111 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_101 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-HADOOP-Build/10916/artifact/patchprocess/diff-checkstyle-hadoop-tools_hadoop-aws.txt JDK v1.7.0_111 Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/10916/testReport/ modules C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/10916/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          stevel@apache.org Steve Loughran added a comment -

          Patch 004 (trying to keep numbers higher). This is Abishek's patch with some slight changes; the one I'll be +1'ing if Yetus is happy

          • cover acceptable suffixes in doc elements of all associated options, in core-defaults and s3/index.md
          • swap order of expected/actual in assertEquals(), as the exception generated on a failure reports parameter 1 as the "expected" value, param 2 as actual.
          Show
          stevel@apache.org Steve Loughran added a comment - Patch 004 (trying to keep numbers higher). This is Abishek's patch with some slight changes; the one I'll be +1'ing if Yetus is happy cover acceptable suffixes in doc elements of all associated options, in core-defaults and s3/index.md swap order of expected/actual in assertEquals() , as the exception generated on a failure reports parameter 1 as the "expected" value, param 2 as actual.
          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 14m 52s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 test4tests 0m 0s 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 mvninstall 9m 6s branch-2 passed
          +1 compile 0m 15s branch-2 passed with JDK v1.8.0_111
          +1 compile 0m 18s branch-2 passed with JDK v1.7.0_111
          +1 checkstyle 0m 19s branch-2 passed
          +1 mvnsite 0m 26s branch-2 passed
          +1 mvneclipse 1m 23s branch-2 passed
          +1 findbugs 0m 36s branch-2 passed
          +1 javadoc 0m 12s branch-2 passed with JDK v1.8.0_111
          +1 javadoc 0m 15s branch-2 passed with JDK v1.7.0_111
          +1 mvninstall 0m 17s the patch passed
          +1 compile 0m 12s the patch passed with JDK v1.8.0_111
          +1 javac 0m 12s the patch passed
          +1 compile 0m 15s the patch passed with JDK v1.7.0_111
          +1 javac 0m 15s the patch passed
          -0 checkstyle 0m 11s hadoop-tools/hadoop-aws: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
          +1 mvnsite 0m 21s the patch passed
          +1 mvneclipse 0m 11s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 0m 41s the patch passed
          +1 javadoc 0m 9s the patch passed with JDK v1.8.0_111
          +1 javadoc 0m 12s the patch passed with JDK v1.7.0_111
          +1 unit 0m 20s hadoop-aws in the patch passed with JDK v1.7.0_111.
          +1 asflicense 0m 18s The patch does not generate ASF License warnings.
          32m 54s



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:b59b8b7
          JIRA Issue HADOOP-13680
          GITHUB PR https://github.com/apache/hadoop/pull/145
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux afa7e5528859 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision branch-2 / e4023f0
          Default Java 1.7.0_111
          Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_111 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_111
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-HADOOP-Build/10931/artifact/patchprocess/diff-checkstyle-hadoop-tools_hadoop-aws.txt
          JDK v1.7.0_111 Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/10931/testReport/
          modules C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/10931/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 14m 52s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. -1 test4tests 0m 0s 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 mvninstall 9m 6s branch-2 passed +1 compile 0m 15s branch-2 passed with JDK v1.8.0_111 +1 compile 0m 18s branch-2 passed with JDK v1.7.0_111 +1 checkstyle 0m 19s branch-2 passed +1 mvnsite 0m 26s branch-2 passed +1 mvneclipse 1m 23s branch-2 passed +1 findbugs 0m 36s branch-2 passed +1 javadoc 0m 12s branch-2 passed with JDK v1.8.0_111 +1 javadoc 0m 15s branch-2 passed with JDK v1.7.0_111 +1 mvninstall 0m 17s the patch passed +1 compile 0m 12s the patch passed with JDK v1.8.0_111 +1 javac 0m 12s the patch passed +1 compile 0m 15s the patch passed with JDK v1.7.0_111 +1 javac 0m 15s the patch passed -0 checkstyle 0m 11s hadoop-tools/hadoop-aws: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2) +1 mvnsite 0m 21s the patch passed +1 mvneclipse 0m 11s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 0m 41s the patch passed +1 javadoc 0m 9s the patch passed with JDK v1.8.0_111 +1 javadoc 0m 12s the patch passed with JDK v1.7.0_111 +1 unit 0m 20s hadoop-aws in the patch passed with JDK v1.7.0_111. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 32m 54s Subsystem Report/Notes Docker Image:yetus/hadoop:b59b8b7 JIRA Issue HADOOP-13680 GITHUB PR https://github.com/apache/hadoop/pull/145 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux afa7e5528859 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision branch-2 / e4023f0 Default Java 1.7.0_111 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_111 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-HADOOP-Build/10931/artifact/patchprocess/diff-checkstyle-hadoop-tools_hadoop-aws.txt JDK v1.7.0_111 Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/10931/testReport/ modules C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/10931/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          stevel@apache.org Steve Loughran added a comment -

          checkstyle is line length:

          ./hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java:189:      readAhead = longByteOption(conf, READAHEAD_RANGE, DEFAULT_READAHEAD_RANGE, 0);: Line is longer than 80 characters (found 84).
          

          I'll fix that on the commit. Local test verified this BTW, S3 Ireland, with the -Pscale profile for scale tests too, as these play with the numbers.

          Show
          stevel@apache.org Steve Loughran added a comment - checkstyle is line length: ./hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java:189: readAhead = longByteOption(conf, READAHEAD_RANGE, DEFAULT_READAHEAD_RANGE, 0);: Line is longer than 80 characters (found 84). I'll fix that on the commit. Local test verified this BTW, S3 Ireland, with the -Pscale profile for scale tests too, as these play with the numbers.
          Hide
          stevel@apache.org Steve Loughran added a comment -

          +1

          committed. Thanks for this!

          Show
          stevel@apache.org Steve Loughran added a comment - +1 committed. Thanks for this!
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10735 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10735/)
          HADOOP-13680. fs.s3a.readahead.range to use getLongBytes. Contributed by (stevel: rev a1761a841e95ef7d2296ac3e40b3a26d97787eab)

          • (edit) hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3AConfiguration.java
          • (edit) hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AUtils.java
          • (edit) hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/index.md
          • (edit) hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java
          • (edit) hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10735 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10735/ ) HADOOP-13680 . fs.s3a.readahead.range to use getLongBytes. Contributed by (stevel: rev a1761a841e95ef7d2296ac3e40b3a26d97787eab) (edit) hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3AConfiguration.java (edit) hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AUtils.java (edit) hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/index.md (edit) hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java (edit) hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user steveloughran closed the pull request at:

          https://github.com/apache/hadoop/pull/145

          Show
          githubbot ASF GitHub Bot added a comment - Github user steveloughran closed the pull request at: https://github.com/apache/hadoop/pull/145
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user abmodi closed the pull request at:

          https://github.com/apache/hadoop/pull/144

          Show
          githubbot ASF GitHub Bot added a comment - Github user abmodi closed the pull request at: https://github.com/apache/hadoop/pull/144

            People

            • Assignee:
              abmodi Abhishek Modi
              Reporter:
              stevel@apache.org Steve Loughran
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development