Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21.0
    • Component/s: io
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      New configuration parameter io.seqfile.local.dir for use by SequenceFile replaces mapred.local.dir.

      Description

      SequenceFile is using "mapred.local.dir". It should not depend on mapred as it is part of the core.

      1. 5219_v1.patch
        1 kB
        Sharad Agarwal

        Issue Links

          Activity

          Hide
          Robert Chansler added a comment -

          Editorial pass over all release notes prior to publication of 0.21.

          Show
          Robert Chansler added a comment - Editorial pass over all release notes prior to publication of 0.21.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-trunk #763 (See http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/763/)
          . Adds a new property io.seqfile.local.dir for use by SequenceFile, which earlier used mapred.local.dir. Contributed by Sharad Agarwal.

          Show
          Hudson added a comment - Integrated in Hadoop-trunk #763 (See http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/763/ ) . Adds a new property io.seqfile.local.dir for use by SequenceFile, which earlier used mapred.local.dir. Contributed by Sharad Agarwal.
          Hide
          Devaraj Das added a comment -

          I just committed this. Thanks, Sharad!

          Show
          Devaraj Das added a comment - I just committed this. Thanks, Sharad!
          Hide
          Sharad Agarwal added a comment -

          test failures are unrelated

          Show
          Sharad Agarwal added a comment - test failures are unrelated
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12400086/5219_v1.patch
          against trunk revision 743816.

          +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 tests are needed for this patch.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs warnings.

          +1 Eclipse classpath. The patch retains Eclipse classpath integrity.

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

          -1 core tests. The patch failed core unit tests.

          -1 contrib tests. The patch failed contrib unit tests.

          Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/artifact/trunk/build/test/checkstyle-errors.html
          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/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/12400086/5219_v1.patch against trunk revision 743816. +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 tests are needed for this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 Eclipse classpath. The patch retains Eclipse classpath integrity. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed core unit tests. -1 contrib tests. The patch failed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3841/console This message is automatically generated.
          Hide
          Doug Cutting added a comment -

          Looks good to me. +1

          Show
          Doug Cutting added a comment - Looks good to me. +1
          Hide
          Sharad Agarwal added a comment -

          straightforward patch.

          Show
          Sharad Agarwal added a comment - straightforward patch.
          Hide
          Doug Cutting added a comment -

          > should we add a property in core-default.xml say "io.seqfile.local.dir" with default value as "$

          {hadoop.tmp.dir}

          /io/local"

          That sounds like a good plan. +1

          Show
          Doug Cutting added a comment - > should we add a property in core-default.xml say "io.seqfile.local.dir" with default value as "$ {hadoop.tmp.dir} /io/local" That sounds like a good plan. +1
          Hide
          Sharad Agarwal added a comment -

          should we add a property in core-default.xml say "io.seqfile.local.dir" with default value as "$

          {hadoop.tmp.dir}

          /io/local"

          Show
          Sharad Agarwal added a comment - should we add a property in core-default.xml say "io.seqfile.local.dir" with default value as "$ {hadoop.tmp.dir} /io/local"

            People

            • Assignee:
              Sharad Agarwal
              Reporter:
              Sharad Agarwal
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development