Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.17.0
    • Fix Version/s: 0.18.0
    • Component/s: fs
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      As of HADOOP-2095, InMemoryFileSystem is no longer used. Its design was optimized for a particular application and it is thus not a good general-purpose RAM-based FileSystem implementation, so it ought to be removed.

      1. 3051.patch
        1 kB
        Devaraj Das
      2. HADOOP-3051.patch
        2 kB
        Doug Cutting

        Issue Links

          Activity

          Hide
          Doug Cutting added a comment -

          Patch to deprecate InMemoryFileSystem.

          Show
          Doug Cutting added a comment - Patch to deprecate InMemoryFileSystem.
          Hide
          Arun C Murthy added a comment -

          +1

          We can fix the test-case which uses the InMemoryFileSystem at a latter point.

          Show
          Arun C Murthy added a comment - +1 We can fix the test-case which uses the InMemoryFileSystem at a latter point.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12383471/HADOOP-3051.patch
          against trunk revision 663487.

          +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 generated 455 javac compiler warnings (more than the trunk's current 452 warnings).

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

          +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/2593/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2593/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2593/artifact/trunk/build/test/checkstyle-errors.html
          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2593/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/12383471/HADOOP-3051.patch against trunk revision 663487. +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 generated 455 javac compiler warnings (more than the trunk's current 452 warnings). +1 findbugs. The patch does not introduce any new Findbugs warnings. +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/2593/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2593/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2593/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2593/console This message is automatically generated.
          Hide
          Doug Cutting added a comment -

          > -1 javac. The applied patch generated 455 javac compiler warnings (more than the trunk's current 452 warnings).

          This is expected, since this patch adds a deprecation.

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

          These same tests are failing for other patches and are unrelated to this patch.

          Show
          Doug Cutting added a comment - > -1 javac. The applied patch generated 455 javac compiler warnings (more than the trunk's current 452 warnings). This is expected, since this patch adds a deprecation. > -1 core tests. The patch failed core unit tests. > -1 contrib tests. The patch failed contrib unit tests. These same tests are failing for other patches and are unrelated to this patch.
          Hide
          Devaraj Das added a comment -

          This patch is w.r.t the trunk.

          Show
          Devaraj Das added a comment - This patch is w.r.t the trunk.
          Hide
          Devaraj Das added a comment -

          Pushing through hudson

          Show
          Devaraj Das added a comment - Pushing through hudson
          Hide
          Owen O'Malley added a comment -

          I just committed this. Thanks, Doug!

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

            People

            • Assignee:
              Doug Cutting
              Reporter:
              Doug Cutting
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development