Hadoop Common
  1. Hadoop Common
  2. HADOOP-4648

Remove ChecksumDistriubtedFileSystem and InMemoryFileSystem

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21.0
    • Component/s: fs
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      Removed obsolete, deprecated subclasses of ChecksumFileSystem (InMemoryFileSystem, ChecksumDistributedFileSystem).

      Description

      We should remove the obsolete Checksum FileSystems.

      1. 4648-0.patch
        11 kB
        Chris Douglas
      2. 4648-1.patch
        26 kB
        Chris Douglas
      3. 4648-2.patch
        30 kB
        Chris Douglas

        Issue Links

          Activity

          Hide
          Tsz Wo Nicholas Sze added a comment -

          Do you also want to remove InMemoryFileSystem here?

          Show
          Tsz Wo Nicholas Sze added a comment - Do you also want to remove InMemoryFileSystem here?
          Hide
          Chris Douglas added a comment -

          Whoops. Thanks, Nicholas.

          Attached a patch removing InMemoryFileSystem

          Show
          Chris Douglas added a comment - Whoops. Thanks, Nicholas. Attached a patch removing InMemoryFileSystem
          Hide
          Tsz Wo Nicholas Sze added a comment -

          +1

          Need release note.

          Show
          Tsz Wo Nicholas Sze added a comment - +1 Need release note.
          Hide
          Chris Douglas added a comment -

          I just committed this.

          Show
          Chris Douglas added a comment - I just committed this.
          Hide
          Chris Douglas added a comment -

          I reverted this. I missed some of the test cases in the last patch.

          Show
          Chris Douglas added a comment - I reverted this. I missed some of the test cases in the last patch.
          Hide
          Hudson added a comment -
          Show
          Hudson added a comment - Integrated in Hadoop-trunk #680 (See http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/680/ )
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12396457/4648-2.patch
          against trunk revision 728676.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 22 new or modified tests.

          +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 core tests. The patch failed core unit tests.

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/artifact/trunk/build/test/checkstyle-errors.html
          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/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/12396457/4648-2.patch against trunk revision 728676. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 22 new or modified tests. +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 core tests. The patch failed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3754/console This message is automatically generated.
          Hide
          Tsz Wo Nicholas Sze added a comment -

          I just committed this. Thanks, Chris!

          Show
          Tsz Wo Nicholas Sze added a comment - I just committed this. Thanks, Chris!
          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.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development