You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Tom White (JIRA)" <ji...@apache.org> on 2009/09/02 08:52:32 UTC

[jira] Created: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Allow caching of filesystem instances to be disabled on a per-instance basis
----------------------------------------------------------------------------

                 Key: HADOOP-6231
                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
             Project: Hadoop Common
          Issue Type: Improvement
          Components: fs
            Reporter: Tom White
            Assignee: Tom White


HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12757430#action_12757430 ] 

Mahadev konar commented on HADOOP-6231:
---------------------------------------

sorry pressed enter too soon,
 I meant "multiple archives in a single jvm wont work"



> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Nigel Daley (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12757388#action_12757388 ] 

Nigel Daley commented on HADOOP-6231:
-------------------------------------

Isn't this really an improvement (not a bug).  Can you write a release note for this?

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6231:
------------------------------

    Fix Version/s: 0.21.0
           Status: Patch Available  (was: Open)

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12750768#action_12750768 ] 

Hadoop QA commented on HADOOP-6231:
-----------------------------------

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12418457/HADOOP-6231-v2.patch
  against trunk revision 810714.

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

    +1 tests included.  The patch appears to include 3 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 release audit.  The applied patch does not increase the total number of release audit warnings.

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

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

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h1.grid.sp2.yahoo.net/4/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h1.grid.sp2.yahoo.net/4/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h1.grid.sp2.yahoo.net/4/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h1.grid.sp2.yahoo.net/4/console

This message is automatically generated.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6231:
------------------------------

    Attachment: HADOOP-6231.patch

A patch that implements this in the way suggested at https://issues.apache.org/jira/browse/HADOOP-6097?focusedCommentId=12745027&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_12745027.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>         Attachments: HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12750311#action_12750311 ] 

Hadoop QA commented on HADOOP-6231:
-----------------------------------

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12418346/HADOOP-6231.patch
  against trunk revision 810384.

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

    +1 tests included.  The patch appears to include 3 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 release audit.  The applied patch does not increase the total number of release audit warnings.

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

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

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/8/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/8/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/8/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/8/console

This message is automatically generated.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>         Attachments: HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6231:
------------------------------

    Attachment: HADOOP-6231-v2.patch

Thanks for the suggestion, Chris. Here's a modified patch.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated HADOOP-6231:
----------------------------------

      Resolution: Fixed
    Hadoop Flags: [Reviewed]
          Status: Resolved  (was: Patch Available)

I just committed this to 0.20 branch. thanks tom and ben!

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12751878#action_12751878 ] 

Hudson commented on HADOOP-6231:
--------------------------------

Integrated in Hadoop-Common-trunk #86 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/86/])
    

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12757429#action_12757429 ] 

Mahadev konar commented on HADOOP-6231:
---------------------------------------

nigel,
  This is really a bug, given that multiple archives in a single wont work without this patch. Please look at HADOOP-6097 for more details.
  I will add a release note on this.

thanks

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12751665#action_12751665 ] 

Hudson commented on HADOOP-6231:
--------------------------------

Integrated in Hadoop-Common-trunk-Commit #13 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/13/])
    . Allow caching of filesystem instances to be disabled on a per-instance basis.


> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated HADOOP-6231:
----------------------------------

    Release Note: Allow a general mechanism to disable the cache on a per filesystem basis by using property fs.<schemename>.impl.disable.cache. eg. fs.har.impl.disable.cache in core-default.xml

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Ben Slusky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ben Slusky updated HADOOP-6231:
-------------------------------

    Attachment: HADOOP-6231-branch-0.20.patch

Patch against the 0.20 branch.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Ben Slusky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ben Slusky reopened HADOOP-6231:
--------------------------------

      Assignee: Ben Slusky  (was: Tom White)

This fixes a bug affecting all releases from 0.18 (when Hadoop archives were introduced). Without this patch, only one Hadoop archive can be opened at a time:

{noformat}
$ hadoop dfs -ls har:///user/knoguchi/test.har har:///user/knoguchi/test2.har
Found 1 items
drw-r--r--   - knoguchi users          0 2009-08-18 18:52 /user/knoguchi/test.har/user
ls: Invalid file name: /user/knoguchi/test2.har in har:///user/knoguchi/test.har

$ hadoop dfs -ls har:///user/knoguchi/test2.har har:///user/knoguchi/test.har
Found 1 items
drw-------   - knoguchi users          0 2009-08-17 19:15 /user/knoguchi/test2.har/user
ls: Invalid file name: /user/knoguchi/test.har in har:///user/knoguchi/test2.har
{noformat}

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12756828#action_12756828 ] 

Mahadev konar commented on HADOOP-6231:
---------------------------------------

   [exec]
     [exec] +1 overall.
     [exec]
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec]
     [exec]     +1 tests included.  The patch appears to include 3 new or modified tests.
     [exec]
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
     [exec]
     [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
     [exec]
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
     [exec]
     [exec]     +1 Eclipse classpath. The patch retains Eclipse classpath integrity.
     [exec]

output of ant test-patch.... am still running ant test and will post the results.... 

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6231:
------------------------------

      Resolution: Fixed
    Hadoop Flags: [Reviewed]
          Status: Resolved  (was: Patch Available)

I've just committed this.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12755516#action_12755516 ] 

Hadoop QA commented on HADOOP-6231:
-----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12419639/HADOOP-6231-branch-0.20.patch
  against trunk revision 815317.

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

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

    -1 patch.  The patch command could not apply the patch.

Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/35/console

This message is automatically generated.

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Ben Slusky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ben Slusky updated HADOOP-6231:
-------------------------------

    Affects Version/s: 0.18.0
                       0.18.1
                       0.18.2
                       0.18.3
                       0.19.0
                       0.19.1
                       0.19.2
                       0.20.0
                       0.20.1
        Fix Version/s:     (was: 0.21.0)
                       0.20.2
           Issue Type: Bug  (was: Improvement)

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1
>            Reporter: Tom White
>            Assignee: Tom White
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Ben Slusky (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ben Slusky updated HADOOP-6231:
-------------------------------

    Hadoop Flags:   (was: [Reviewed])
          Status: Patch Available  (was: Reopened)

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.20.1, 0.20.0, 0.19.2, 0.19.1, 0.19.0, 0.18.3, 0.18.2, 0.18.1, 0.18.0
>            Reporter: Tom White
>            Assignee: Ben Slusky
>             Fix For: 0.20.2
>
>         Attachments: HADOOP-6231-branch-0.20.patch, HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6231:
------------------------------

    Status: Patch Available  (was: Open)

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>         Attachments: HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6231:
------------------------------

    Status: Open  (was: Patch Available)

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>         Attachments: HADOOP-6231-v2.patch, HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HADOOP-6231) Allow caching of filesystem instances to be disabled on a per-instance basis

Posted by "Chris Douglas (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12750318#action_12750318 ] 

Chris Douglas commented on HADOOP-6231:
---------------------------------------

The unit test should use JUnit4 conventions rather than JUnit3, but aside from that the fix looks good

> Allow caching of filesystem instances to be disabled on a per-instance basis
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-6231
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6231
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Tom White
>            Assignee: Tom White
>         Attachments: HADOOP-6231.patch
>
>
> HAR filesystem instances should not be cached, so this JIRA seeks to provide a general mechanism for disabling the cache on a per-filesystem basis. (Carried over from HADOOP-6097.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.