Hadoop Common
  1. Hadoop Common
  2. HADOOP-6991

SequenceFile::Reader discards length for files, does not call openFile

    Details

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

      Description

      While the sorting and merging in SequenceFile is deprecated and openFile is archaic, the semantics should remain consistent.

      1. h-6991-2.patch
        2 kB
        Owen O'Malley
      2. 6991-1.patch
        2 kB
        Chris Douglas
      3. 6991-0.patch
        2 kB
        Chris Douglas

        Issue Links

          Activity

          Hide
          Chris Douglas added a comment -

          Patch on top of HADOOP-6984

          Show
          Chris Douglas added a comment - Patch on top of HADOOP-6984
          Hide
          Owen O'Malley added a comment -

          Updated Chris' patch with the test case moved from MapRed and turning on overwrite.

          Show
          Owen O'Malley added a comment - Updated Chris' patch with the test case moved from MapRed and turning on overwrite.
          Hide
          Owen O'Malley added a comment -

          Forgot test case.

          Show
          Owen O'Malley added a comment - Forgot test case.
          Hide
          Chris Douglas added a comment -

          +1

          Ran the SequenceFile* unit tests in MapReduce, all passed.

          Show
          Chris Douglas added a comment - +1 Ran the SequenceFile* unit tests in MapReduce, all passed.
          Hide
          Owen O'Malley added a comment -

          I just committed this. Thanks, Chris!

          Show
          Owen O'Malley added a comment - I just committed this. Thanks, Chris!
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #407 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/407/)
          HADOOP-6991. Fix SequenceFile::Reader to honor file lengths and call
          openFile (cdouglas via omalley)

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #407 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/407/ ) HADOOP-6991 . Fix SequenceFile::Reader to honor file lengths and call openFile (cdouglas via omalley)
          Hide
          Hudson added a comment -

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

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

          Integrated in Hadoop-Common-trunk #496 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/496/)
          HADOOP-6991. Fix SequenceFile::Reader to honor file lengths and call
          openFile (cdouglas via omalley)
          HADOOP-6991. Move testcase from mapreduce to common, since it tests
          SequenceFile.

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk #496 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/496/ ) HADOOP-6991 . Fix SequenceFile::Reader to honor file lengths and call openFile (cdouglas via omalley) HADOOP-6991 . Move testcase from mapreduce to common, since it tests SequenceFile.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/)

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/ )

            People

            • Assignee:
              Chris Douglas
              Reporter:
              Chris Douglas
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development